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

Update changelog add upcoming #675

Merged
merged 4 commits into from
Mar 9, 2022
Merged

Conversation

maurolacy
Copy link
Contributor

Adds a -u|--upcoming-tag <tag> option to update_changelog.sh, so that a new entry is created in CHANGELOG.md for the upcoming version tag.

Copy link
Contributor

@ueco-jb ueco-jb left a comment

Choose a reason for hiding this comment

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

lgtm

Comment on lines +72 to +73
sed -i "s+\[Full Changelog\](https://github.com/CosmWasm/cw-plus/compare/\(.*\)\.\.\.HEAD)+[Full Changelog](https://github.com/CosmWasm/cw-plus/compare/$UPCOMING_TAG...HEAD)\n\n## [$UPCOMING_TAG](https://github.com/CosmWasm/cw-plus/tree/$UPCOMING_TAG) ($TODAY)\n\n[Full Changelog](https://github.com/CosmWasm/cw-plus/compare/\1...$UPCOMING_TAG)+" CHANGELOG.md
fi
Copy link
Contributor

Choose a reason for hiding this comment

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

Just an idea, you could try to add some validator here that sed actually worked (in case anything happens and for example changelog format slightly changes)

Copy link
Contributor Author

@maurolacy maurolacy Mar 9, 2022

Choose a reason for hiding this comment

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

That's an ugly line indeed. If I enable shellcheck here, which I tried, it complains about many issues with this script, and with abundant whining on this line in particular.

Will try to improve on it. Perhaps adding shellcheck, fixing some stuff, and silencing some other warnings.

On the other hand, if sed fails to match here (basically because there's no ...HEAD tag in the changelog, nothing happens.

@maurolacy maurolacy merged commit 2fe1f9d into main Mar 9, 2022
@maurolacy maurolacy deleted the update_changelog-add-upcoming branch March 9, 2022 15:14
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