Skip to content

fix(release): run mk-release.py before macOS DRE build - #1954

Merged
pietrodimarco-dfinity merged 3 commits into
mainfrom
pmarco/fix-canister-path-filter
Apr 1, 2026
Merged

fix(release): run mk-release.py before macOS DRE build#1954
pietrodimarco-dfinity merged 3 commits into
mainfrom
pmarco/fix-canister-path-filter

Conversation

@pietrodimarco-dfinity

@pietrodimarco-dfinity pietrodimarco-dfinity commented Apr 1, 2026

Copy link
Copy Markdown
Contributor

Summary

The tag-triggered Release DRE binary workflow builds DRE on macOS after checkout, but the in-repo version metadata was not updated before cargo build. This adds the same Update tags in code step used elsewhere: run bin/mk-release.py with the pushed tag (${GITHUB_REF#refs/tags/}) so the binary is built with the correct release version.

…pes from release notes

The canister path filter regex was excluding changes to files like
`canister_http.rs` and `management_canister_types/`, which are core
replica components. Add `_http` and `_types` to the negative lookahead
so these paths are no longer filtered out.

Made-with: Cursor
Tag-triggered releases need version strings updated in-tree before
cargo build; align the macOS job with that step.

Made-with: Cursor
@pietrodimarco-dfinity
pietrodimarco-dfinity requested a review from a team as a code owner April 1, 2026 14:34
@pietrodimarco-dfinity
pietrodimarco-dfinity enabled auto-merge (squash) April 1, 2026 14:37
@pietrodimarco-dfinity
pietrodimarco-dfinity merged commit 2b16828 into main Apr 1, 2026
10 of 12 checks passed
@pietrodimarco-dfinity
pietrodimarco-dfinity deleted the pmarco/fix-canister-path-filter branch April 1, 2026 14:53
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.

2 participants