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(governance-tools): Pretty add-release-to-changelog.sh output. #3417

Merged

Conversation

daniel-wong-dfinity-org
Copy link
Contributor

@daniel-wong-dfinity-org daniel-wong-dfinity-org commented Jan 13, 2025

More Precisely, What

Added a success message.

Added emojis.

Changed informational messages from purple to cyan. I think dark grey would be best, but we don't (yet) have a function for that; cyan seems like the next best thing.

Why

Silence is a highly ambiguous way to communicate.

Explicit is better than implicit.
--Zen of Python

@github-actions github-actions bot added the feat label Jan 13, 2025
@daniel-wong-dfinity-org daniel-wong-dfinity-org force-pushed the pretty-add-release-to-changelog-daniel-wong branch from 781e3fc to 5ff8d89 Compare January 13, 2025 12:27
@daniel-wong-dfinity-org daniel-wong-dfinity-org marked this pull request as ready for review January 13, 2025 12:33
@daniel-wong-dfinity-org daniel-wong-dfinity-org requested a review from a team as a code owner January 13, 2025 12:33
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.

@daniel-wong-dfinity-org daniel-wong-dfinity-org changed the base branch from master to add-entries-to-sns-changelog-daniel-wong January 13, 2025 13:42
@daniel-wong-dfinity-org daniel-wong-dfinity-org changed the title feat(governance-tools): Pretty add release to changelog. feat(governance-tools): Pretty add-release-to-changelog.sh output. Jan 13, 2025
Base automatically changed from add-entries-to-sns-changelog-daniel-wong to master January 14, 2025 16:37
@daniel-wong-dfinity-org daniel-wong-dfinity-org force-pushed the pretty-add-release-to-changelog-daniel-wong branch from 5ff8d89 to ed0a022 Compare January 14, 2025 16:43
@daniel-wong-dfinity-org daniel-wong-dfinity-org added this pull request to the merge queue Jan 14, 2025
@daniel-wong-dfinity-org daniel-wong-dfinity-org removed this pull request from the merge queue due to a manual request Jan 14, 2025
@daniel-wong-dfinity-org daniel-wong-dfinity-org added this pull request to the merge queue Jan 14, 2025
Merged via the queue into master with commit f72bd1c Jan 14, 2025
25 checks passed
@daniel-wong-dfinity-org daniel-wong-dfinity-org deleted the pretty-add-release-to-changelog-daniel-wong branch January 14, 2025 17:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants