Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Move SCR definition to variables. #302

Merged
merged 7 commits into from
Dec 10, 2020
Merged

Move SCR definition to variables. #302

merged 7 commits into from
Dec 10, 2020

Conversation

jaroslawgajewski
Copy link
Contributor

Move SCR definition to variables.

Move SCR definition to variables.
Copy link
Member

@bharathkkb bharathkkb left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi @jaroslawgajewski
Thanks for the PR! Could you also regenerate docs by running make docker_generate_docs

@jaroslawgajewski
Copy link
Contributor Author

Hi @bharathkkb
I've regenerated documentation again but still failing for lint checks. Can you have a look what is that related to? I was trying to remove HTLM entries from readme manually but it seem scripts are putting it back during doc generation.

@rjerrems
Copy link
Collaborator

rjerrems commented Dec 9, 2020

Hi @jaroslawgajewski - it looks like you need to run a terraform fmt

gcr.io/cloud-foundation-cicd/cft/developer-tools:0.13
Checking for documentation generation
Checking for trailing whitespace
Checking for missing newline at end of file
Running shellcheck
Checking file headers
Running flake8
Attempting to download /workspace/test/bundle.hcl bundle.
/tmp/bundler /workspace
Failed to read config: At 6:12: root.providers.google: not an object type for struct (*ast.ListType)
unzip: can't open terraform_*.zip[.zip]
/workspace
Running terraform fmt
0-bootstrap/variables.tf
--- old/0-bootstrap/variables.tf
+++ new/0-bootstrap/variables.tf
@@ -78,7 +78,7 @@
 variable "cloud_source_repos" {
   description = "List of Cloud Source Reposiories created during bootrstap project build stage"
   type        = list(string)
-  default     = ["gcp-bootstrap","gcp-org","gcp-environments","gcp-networks","gcp-projects"]
+  default     = ["gcp-bootstrap", "gcp-org", "gcp-environments", "gcp-networks", "gcp-projects"]
 }
 
 /* ----------------------------------------
Error: terraform fmt failed with exit code 3
Check the output for diffs and correct using terraform fmt <dir>

Also FYI, you can run linting yourself locally by running make docker_test_lint

@bharathkkb
Copy link
Member

/gcbrun

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants