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
Decouple clean build CI job from prod build #1814
Conversation
This PR splits the canister_tests clean build CI job into two parts: the clean build & the hash check. This allows to start the clean build immediately instead of just after the prod build. Also, the clean build now really builds the prod build from scratch, including the dev build that's hashed into it due to dfx deps (if applicable, i.e. only release builds).
6a9bc9a
to
3082c22
Compare
3082c22
to
282d8b3
Compare
@@ -70,7 +70,6 @@ jobs: | |||
|
|||
# Since the release build check is a scheduled job, a failure won't be shown on any | |||
# PR status. To notify the team, we send a message to our Slack channel on failure. | |||
- uses: actions/checkout@v3 # checks out the latest slack action |
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.
This checkout action has been removed because the post-run actions of the check-build
action crash with out-of index if the number of steps in that action has changed since the last release.
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.
LGTM, thanks!
This PR splits the canister_tests clean build CI job into two parts: the clean build & the hash check.
This allows to start the clean build immediately instead of just after the prod build.
Also, the clean build now really builds
the prod build from scratch, including the dev build that's hashed into it due to dfx deps (if applicable, i.e. only release builds).
Link to successful release build check run: https://github.com/dfinity/internet-identity/actions/runs/5975993428
🟡 Some screens were changed