Skip to content

fix(github-action): Analyze checkout & upload bug - #519

Merged
dannyvassallo merged 9 commits into
aws-cloudformation:mainfrom
dannyvassallo:analyze-checkout-bug
Jun 7, 2024
Merged

dannyvassallo merged 9 commits into
aws-cloudformation:mainfrom
dannyvassallo:analyze-checkout-bug

Conversation

@dannyvassallo

Copy link
Copy Markdown
Contributor

Issue #, if available:

N/A

Description of changes:

Fixes missing SHA and ref for checkoutPublicRepository & uploadCodeScan


By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license

@joshfried-aws joshfried-aws left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

nit: title should replace the word bug with fix according to the conventional commits spec

Comment thread action/README.md
steps:
- name: CloudFormation Guard validate
uses: aws-cloudformation/cloudformation-guard@action-v0.0.2
uses: aws-cloudformation/cloudformation-guard@action-v0.0.3

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

nit: i assume theres a way we can make this less tedious when bumping the version?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

we could just x.x.x it and leave it at that or template it.

@dannyvassallo dannyvassallo changed the title bug(github-action): Analyze checkout & upload bug fix(github-action): Analyze checkout & upload bug Jun 7, 2024
@dannyvassallo
dannyvassallo merged commit 9509936 into aws-cloudformation:main Jun 7, 2024
joshfried-aws pushed a commit to joshfried-aws/cloudformation-guard that referenced this pull request Nov 22, 2024
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.

3 participants