Merge pull request #11 from olhado/fixing-null-arn

Fixing null arn

Delete temp_provider.tf (#9)

Removed erroneous test file.

Match to docs (#2)

* Removed an empty file, and added a `.gitignore` file to not track artifacts of running `terraform init/plan/apply`

* Ran `terraform 0.12-compatibility` to do a basic 0.11->0.12 conversion.

* Put proper years in license

* Changed input variable structure. This allows for cleaner extension of config if we decide to cover more cloud providers in the future.

* Removed commented out old parameters. Added comments.

* Updated README to reflect new input variable names/structure. Renamed one varable back to its original name. Deleted an obsolete screenshot from the documentation/README.

* Fix a typo in the README and a typo in the variable definition

* Use proper 0.12 syntax

* Fix variable types to string

* Fix more 0.12 syntax issues

* Restructured configuration due to limitations in terraform variable definition syntax.

See: hashicorp/terraform#19898

* Fix a place I didn't correctly rename a variable reference

* Updated code comments and README with correct variable information, and a few typo fixes along the way.

* Add dependency documention for required Terraform providers.

* Fix some docs, and add the defaults for the optional settings

* Remove largely unneeded variable. This also aligns the terraform module more closely to the CloudFormation template.

Fix role policy for S3 bucket access