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

Add Indeni Cloudrail #1047

Merged
merged 6 commits into from
Oct 4, 2021
Merged

Add Indeni Cloudrail #1047

merged 6 commits into from
Oct 4, 2021

Conversation

yi2020
Copy link
Contributor

@yi2020 yi2020 commented Aug 17, 2021

NOTE: Not clear on how to test this. Our GitHub Action is well tested, of course.

@yi2020 yi2020 requested review from a team as code owners August 17, 2021 14:32
@github-actions github-actions bot added the code-scanning Related to workflows that show on the Code Scanning setup page label Aug 17, 2021
@actions actions deleted a comment Sep 8, 2021
@actions actions deleted a comment from Ashleycho35 Sep 27, 2021
@actions actions deleted a comment Sep 27, 2021
Copy link
Contributor

@nickfyson nickfyson left a comment

Choose a reason for hiding this comment

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

Thanks for this PR, and apologies it's taken a while for you to get a review!

The tile is looking good...

In line with our other workflows, though, I've proposed a few edits to the description (to keep it a bit shorter).

Other than that, just a change to add full SHA pinning.

code-scanning/cloudrail.yml Outdated Show resolved Hide resolved
code-scanning/properties/cloudrail.properties.json Outdated Show resolved Hide resolved
@yi2020
Copy link
Contributor Author

yi2020 commented Oct 4, 2021

@nickfyson thank you for the comments and apologize for the delay in my response, please take another look now.

Copy link
Contributor

@nickfyson nickfyson left a comment

Choose a reason for hiding this comment

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

LGTM

@nickfyson nickfyson merged commit 5d273fb into actions:main Oct 4, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
code-scanning Related to workflows that show on the Code Scanning setup page
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants