Skip to content

Conversation

@dannyvassallo
Copy link
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

Copy link
Contributor

@joshfried-aws joshfried-aws left a comment

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

steps:
- name: CloudFormation Guard validate
uses: aws-cloudformation/[email protected].2
uses: aws-cloudformation/[email protected].3
Copy link
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
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
…#519)

Fix publicCheckout & uploadCodeScan on schedule events
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