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

feat(nervous-system): release runscript automatically creates changelog PRs #3806

Merged
merged 1 commit into from
Feb 6, 2025

Conversation

anchpop
Copy link
Contributor

@anchpop anchpop commented Feb 5, 2025

This is the final step in the release process. Now everything is automated!

Here is an example of a PR it creates: #3807

To create the PR, we use gh, which we assert you have on your computer before the first step. (This ensures that you don't get to the last step and then hit an error which would be slightly annoying to recover from. Instead, you get the error right away, and it should be smooth sailing after that.)

The intention of this change is that you create the PR to update the changelog right when you create the proposals.

Yes, we can optimistically assume that the proposal will pass. If it doesn't, we can revert the CHANGELOG.md. Branch predict FTW.

A consequence of is that the changelog now shows the date of proposal submission, rather than execution.

← Previous PR

@anchpop anchpop requested a review from a team as a code owner February 5, 2025 20:09
@anchpop anchpop changed the title @anchpop/create changelog prs feat(nervous-system): release runscript automatically creates changelog PRs Feb 5, 2025
Copy link
Contributor

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

If this pull request affects the behavior of any canister owned by
the Governance team, remember to update the corresponding
unreleased_changes.md file(s).

To acknowldge this reminder (and unblock the PR), dismiss this
code review by going to the bottom of the pull request page, and
supply one of the following reasons:

  1. Done.

  2. No canister behavior changes.

@github-actions github-actions bot added the feat label Feb 5, 2025
@anchpop anchpop force-pushed the @anchpop/create-changelog-prs branch 2 times, most recently from 1293d2c to 3db6f40 Compare February 5, 2025 20:16
@anchpop anchpop dismissed github-actions[bot]’s stale review February 5, 2025 20:29

No canister behavior changes.

@anchpop anchpop force-pushed the @anchpop/create-changelog-prs branch from 3db6f40 to c242bb2 Compare February 5, 2025 20:30
@anchpop anchpop force-pushed the @anchpop/utils-file branch from 4e9b9e1 to 5734b25 Compare February 6, 2025 03:34
@anchpop anchpop force-pushed the @anchpop/create-changelog-prs branch 2 times, most recently from 9d29b35 to 71bd94e Compare February 6, 2025 03:57
@anchpop anchpop force-pushed the @anchpop/utils-file branch 2 times, most recently from cb374fd to 53de069 Compare February 6, 2025 04:18
@anchpop anchpop force-pushed the @anchpop/create-changelog-prs branch 2 times, most recently from e7b0579 to b3c25d9 Compare February 6, 2025 07:04
@anchpop anchpop force-pushed the @anchpop/utils-file branch 2 times, most recently from d3d5d48 to 96e2be9 Compare February 6, 2025 07:31
github-merge-queue bot pushed a commit that referenced this pull request Feb 6, 2025
This improves code organization because now everything in main.rs is the
essential logic (rather than helpers that you usually don't need to care
about)

[← Previous PR](#3794) | [Next PR
→](#3806)
Base automatically changed from @anchpop/utils-file to master February 6, 2025 07:59
@anchpop anchpop force-pushed the @anchpop/create-changelog-prs branch from b3c25d9 to 1bdb00e Compare February 6, 2025 08:01
@anchpop anchpop enabled auto-merge February 6, 2025 08:01
@anchpop anchpop added this pull request to the merge queue Feb 6, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Feb 6, 2025
@anchpop anchpop added this pull request to the merge queue Feb 6, 2025
Merged via the queue into master with commit cc2a047 Feb 6, 2025
25 checks passed
@anchpop anchpop deleted the @anchpop/create-changelog-prs branch February 6, 2025 16:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants