-
Notifications
You must be signed in to change notification settings - Fork 32
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
Update control 1.1.6 #44
Update control 1.1.6 #44
Conversation
Signed-off-by: Frederick Witty <[email protected]> Signed-off-by: Mathieu Fortin <[email protected]>
Signed-off-by: Frederick Witty <[email protected]> Signed-off-by: Mathieu Fortin <[email protected]>
Signed-off-by: Frederick Witty <[email protected]> Signed-off-by: Mathieu Fortin <[email protected]>
Signed-off-by: Frederick Witty <[email protected]> Signed-off-by: Mathieu Fortin <[email protected]>
Signed-off-by: Frederick Witty <[email protected]> Signed-off-by: Mathieu Fortin <[email protected]>
Signed-off-by: Frederick Witty <[email protected]> Signed-off-by: Mathieu Fortin <[email protected]>
Signed-off-by: Frederick Witty <[email protected]> Signed-off-by: Mathieu Fortin <[email protected]>
Signed-off-by: Frederick Witty <[email protected]> Signed-off-by: Mathieu Fortin <[email protected]>
Signed-off-by: Frederick Witty <[email protected]> Signed-off-by: Mathieu Fortin <[email protected]>
Signed-off-by: Frederick Witty <[email protected]> Signed-off-by: Mathieu Fortin <[email protected]>
Signed-off-by: Frederick Witty <[email protected]> Signed-off-by: Mathieu Fortin <[email protected]>
Signed-off-by: Frederick Witty <[email protected]> Signed-off-by: Mathieu Fortin <[email protected]>
Signed-off-by: Frederick Witty <[email protected]> Signed-off-by: Mathieu Fortin <[email protected]>
Signed-off-by: Frederick Witty <[email protected]> Signed-off-by: Mathieu Fortin <[email protected]>
Signed-off-by: Frederick Witty <[email protected]> Signed-off-by: Mathieu Fortin <[email protected]>
Signed-off-by: Frederick Witty <[email protected]> Signed-off-by: Mathieu Fortin <[email protected]>
Signed-off-by: Frederick Witty <[email protected]> Signed-off-by: Mathieu Fortin <[email protected]>
Signed-off-by: Frederick Witty <[email protected]> Signed-off-by: Mathieu Fortin <[email protected]>
Signed-off-by: Frederick Witty <[email protected]> Signed-off-by: Mathieu Fortin <[email protected]>
Signed-off-by: Frederick Witty <[email protected]> Signed-off-by: Mathieu Fortin <[email protected]>
Signed-off-by: Frederick Witty <[email protected]> Signed-off-by: Mathieu Fortin <[email protected]>
Signed-off-by: Frederick Witty <[email protected]> Signed-off-by: Mathieu Fortin <[email protected]>
Signed-off-by: Mathieu Fortin <[email protected]>
Signed-off-by: Mathieu Fortin <[email protected]>
Signed-off-by: Mathieu Fortin <[email protected]>
Signed-off-by: Mathieu Fortin <[email protected]>
Signed-off-by: Mathieu Fortin <[email protected]>
Signed-off-by: Mathieu Fortin <[email protected]>
Signed-off-by: Mathieu Fortin <[email protected]>
Signed-off-by: Frederick Witty <[email protected]> Signed-off-by: Mathieu Fortin <[email protected]>
Signed-off-by: Frederick Witty <[email protected]> Signed-off-by: Mathieu Fortin <[email protected]>
bad5e91
to
ba2412e
Compare
@frederickw082922 can you restart this test build ? |
@mfortin Thank you for the PR! Do not see the re-run workflow button on the validation. Not sure if actions/checkout@v3 has anything to do with it since there is an actions/checkout@v4. CC: @MrSteve81 |
@frederickw082922 I applied a simple commit to do the rebuild, it worked... |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Approved! Thank you @mfortin ! Good find! CC: @MrSteve81
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Approved
Signed-off-by: fortinm <[email protected]>
Signed-off-by: Math Fortin <[email protected]>
Signed-off-by: Math Fortin <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you @mfortin !
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Approved
Overall Review of Changes:
Control 1.1.6 can't be applied right now
Issue Fixes:
No issues created
Enhancements:
Enforces control id 1.1.6
How has this been tested?:
Locally, on an EC2 instance.