-
Notifications
You must be signed in to change notification settings - Fork 0
Feature/build testing #483
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
Conversation
193e039
to
b4c6e22
Compare
The merge-base changed after approval.
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.
Happy with the changes
The merge-base changed after approval.
The merge-base changed after approval.
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.
Happy with all the changes
The merge-base changed after approval.
2ba8a85
to
879e23c
Compare
PR Type
Description
Closes #468. The workflow and updated CONTRIBUTING wording are copied with very minor edits from Coreax.
Note that this PR is dependent on/closes #481, as it contains commits from that PR - this was done intentionally to avoid merge conflicts in CONTRIBUTING.md
How Has This Been Tested?
The workflow actually hasn't been tested here, but has been copied (with very minor edits) from Coreax - see gchq/coreax#858.
Does this PR introduce a breaking change?
No
Screenshots
N/A
Checklist before requesting a review