feat: generate changelog from PRs #587
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
Preview link ready! Built with commit af9be8e https://deploy-preview-587--cmu-delphi-covidcast.netlify.app |
|
Yes! This works great. Is there an easy way to also link in associated issues? I'm thinking it'd be nice to have something like this:
|
|
as mentioned in slack, I haven't found a way to get the linked issues for an PR (as listed in the side panel of an PR). One could parse out the |
Actually, I think this a reasonable thing to do. Regex for |
|
then it will generate # Release 1.9.3
## New Features
- [#543](https://github.com/cmu-delphi/www-covidcast/pull/543) (re: [#482](https://github.com/cmu-delphi/www-covidcast/issues/482)) Enforce county geo type
- [#558](https://github.com/cmu-delphi/www-covidcast/pull/558) (re: [#553](https://github.com/cmu-delphi/www-covidcast/issues/553)) Update population data based on indicator data
- [#530](https://github.com/cmu-delphi/www-covidcast/pull/530) (re: [#522](https://github.com/cmu-delphi/www-covidcast/issues/522)) Generate signal constants
- [#526](https://github.com/cmu-delphi/www-covidcast/pull/526) (re: [#512](https://github.com/cmu-delphi/www-covidcast/issues/512)) Add background to map title for visibility
- [#545](https://github.com/cmu-delphi/www-covidcast/pull/545) (re: [#478](https://github.com/cmu-delphi/www-covidcast/issues/478)) Add date shortcuts
- [#551](https://github.com/cmu-delphi/www-covidcast/pull/551) (re: [#493](https://github.com/cmu-delphi/www-covidcast/issues/493)) Enforce 0 is no bubble
## Bug-fixes
- [#541](https://github.com/cmu-delphi/www-covidcast/pull/541) (re: [#528](https://github.com/cmu-delphi/www-covidcast/issues/528)) Add aria label to map
- [#460](https://github.com/cmu-delphi/www-covidcast/pull/460) (re: [#539](https://github.com/cmu-delphi/www-covidcast/issues/539)) Add missing county level to quidel
- [#535](https://github.com/cmu-delphi/www-covidcast/pull/535) (re: [#534](https://github.com/cmu-delphi/www-covidcast/issues/534)) Incomplete imputation in detail view
- [#514](https://github.com/cmu-delphi/www-covidcast/pull/514) (re: [#513](https://github.com/cmu-delphi/www-covidcast/issues/513)) Set aria option for no-uislider
- [#515](https://github.com/cmu-delphi/www-covidcast/pull/515) Fix vega warning
## Others
- [#552](https://github.com/cmu-delphi/www-covidcast/pull/552) Use local maxima for top 10
- [#571](https://github.com/cmu-delphi/www-covidcast/pull/571) (re: [#477](https://github.com/cmu-delphi/www-covidcast/issues/477)) Making toggle selection darker
- [#574](https://github.com/cmu-delphi/www-covidcast/pull/574) (re: [#563](https://github.com/cmu-delphi/www-covidcast/issues/563)) Add day ticks and week gridlines for Detail view.
- [#524](https://github.com/cmu-delphi/www-covidcast/pull/524) (re: [#518](https://github.com/cmu-delphi/www-covidcast/issues/518), [#517](https://github.com/cmu-delphi/www-covidcast/issues/517)) Re-enable side panel toggle
- [#525](https://github.com/cmu-delphi/www-covidcast/pull/525) (re: [#519](https://github.com/cmu-delphi/www-covidcast/issues/519), [#516](https://github.com/cmu-delphi/www-covidcast/issues/516)) Search icon and na hint
- [#542](https://github.com/cmu-delphi/www-covidcast/pull/542) (re: [#532](https://github.com/cmu-delphi/www-covidcast/issues/532), [#533](https://github.com/cmu-delphi/www-covidcast/issues/533), [#483](https://github.com/cmu-delphi/www-covidcast/issues/483), [#521](https://github.com/cmu-delphi/www-covidcast/issues/521)) Tune export data tab
- [#544](https://github.com/cmu-delphi/www-covidcast/pull/544) (re: [#540](https://github.com/cmu-delphi/www-covidcast/issues/540), [#538](https://github.com/cmu-delphi/www-covidcast/issues/538)) Coverage info
- [#566](https://github.com/cmu-delphi/www-covidcast/pull/566) Improve vega highlight performance
- [#568](https://github.com/cmu-delphi/www-covidcast/pull/568) (re: [#563](https://github.com/cmu-delphi/www-covidcast/issues/563)) Dlaliberte/#563
- [#546](https://github.com/cmu-delphi/www-covidcast/pull/546) (re: [#540](https://github.com/cmu-delphi/www-covidcast/issues/540)) Single location view
- [#556](https://github.com/cmu-delphi/www-covidcast/pull/556) (re: [#484](https://github.com/cmu-delphi/www-covidcast/issues/484)) Add download button to detail views
- [#523](https://github.com/cmu-delphi/www-covidcast/pull/523) Sync main -> dev
- [#548](https://github.com/cmu-delphi/www-covidcast/pull/548) (re: [#520](https://github.com/cmu-delphi/www-covidcast/issues/520)) Change max mobile width for mode nav buttons |
|
But there could be multiple fixes that relate to the same issue, pulled in
separately. Only the last one that closes the issue will identify the
issue.
Could we add to the pull request template a line for "issue: "?
…On Thu, Oct 22, 2020 at 3:02 PM Chris Scott ***@***.***> wrote:
One could parse out the closes out of the PR body but that isn't reliable.
Actually, I think this a reasonable thing to do. Regex for \#[0-9]+ from
the PR description and simply pull those out. Then you'd see something like Fixes
#587, related to #134, as we discussed in #444 -> #123 (re: #587, #134,
#444). Thoughts?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
<#587 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAELGDOCZ6DSHOV2JMO4HMTSMB6TPANCNFSM4S3SHMUA>
.
--
Daniel LaLiberte <https://plus.google.com/100631381223468223275?prsrc=2>
dlaliberte@Google.com <dlaliberte@google.com> Cambridge MA
|
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Prerequisites:
devbranchdevSummary
adds a script
npm run changelogthat will update aCHANGELOG.mdfile based on the PRs merged between the current checkout out version and the latest main state.when running right now it will look like:
@tildechris is that what you imagined?