-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
Make it easier to release a new version of CONSUL #3866
Conversation
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.
Awesome idea! 🎉 It's funny how we can live with a complicated setup for years and just assume it's normal 😅.
I've left a couple of comments.
Gemfile
Outdated
@@ -93,6 +93,7 @@ group :development do | |||
gem "capistrano3-delayed-job", "~> 1.7.3" | |||
gem "capistrano3-puma", "~> 4.0.0" | |||
gem "erb_lint", require: false | |||
gem "github_changelog_generator" |
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.
For consistency, how about adding ", ~> 1.15.0"
?
Rakefile
Outdated
config.future_release = "1.1.0" | ||
config.base = "#{Rails.root}/CHANGELOG.md" | ||
config.token = Rails.application.secrets.github_changelog_token | ||
config.max_issues = 1 |
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.
In the past, we had max_issues = 1
in the Rakefile and max_issues = 0
in the .github_changelog_generator
file. I don't know which way has precedence 🤔. Is it this one?
With this change we no longer need the branch `changelog` and it will be easier to release new versions of CONSUL.
We only include pull requests, so checking issues isn't useful and takes time.
We were removing the authors manually after they were generated.
80dd85e
to
96be43a
Compare
…sion Make it easier to release a new version of CONSUL
Objectives
With this change we no longer need the branch
changelog
and it will be easier to release new versions of CONSUL.Note
The wiki page about releasing a new version needs to be adjusted