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

Features/reusable workflows and doc #3

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

IGForeau
Copy link
Contributor

Split the main PR workflow into several independant reusable workflows. Said reusable workflows may be copied in a independant repository to allow for parallel maintainance of several repositories' workflows.
In such a case, the new on_pull_request.yml workflow should be updated to point toward the reference repository (private repositories will need specific configuration to enable sharing, see https://docs.github.com/en/actions/using-workflows/reusing-workflows#access-to-reusable-workflows )

Also adds a job that ensures that the documentation is up-to-date. Part of the shell sheganigans occuring in the step Prepare PR message is due to the fact that the terraform-docs step adds updated files to staging even if it is not configured to commit them. The doc workflow call in on_pull_request.yml may also be set to commit documentation changes by setting the commit input to true.

Note that this PR also includes changes from #1

* Add Google auth file if the corresponding secret is set
* Remove terraform plan from estimation job in
  check-secure-estimate workflow
* Remove AWS-related auth env vars in check-secure-estimate workflow
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.

1 participant