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

updated tf validate workflow to latest version of actions #22

Merged
merged 2 commits into from
Sep 20, 2024

Conversation

douglas-f
Copy link
Member

Pull Request Checklist

Take your time to read through these, don't just click through them. Only check the boxes if they apply.

Admin

Types of changes

  • 🐛 Bug fix (non-breaking change which fixes an issue)
  • ❇️ New feature (non-breaking change which adds functionality)
  • 💥 Breaking change (fix or feature that would cause existing functionality to change)

Testing

  • Required: I have tested the proposed changes to code (i.e. packer build, terraform apply), and they are working.
  • Required: All GitHub Actions ran successfully. If they didn't, I left a note in the description to address this.
  • Optional: I have already applied/deployed the changes in my environment.

Please check where this code has been tested:

  • Locally
  • Customer Environment
  • Coalfire Sandbox Environment (AWS GovCloud, Azure Gov, GCP, etc.)

Documentation

  • Optional, recommended: I have updated README.md.
  • Optional, recommended: I have updated documentation in Confluence/Organization's relevant Wiki.
  • Optional, recommended: I left comments in-line in the code to help others understand the changes.

Tagging / Assigning

  • Required: I have tagged a Reviewer(s).
  • Required: I have tagged an Assignee(s).
  • Optional: I have tagged relevant stakeholders in a Comment.

@douglas-f douglas-f merged commit 2b2cb90 into main Sep 20, 2024
0 of 2 checks passed
@douglas-f douglas-f deleted the version_update branch September 20, 2024 22:35
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