Conversation
|
I like it! |
| all.Changed.push(message); | ||
| } else if(filename.endsWith('_fix.md')) { | ||
| all.Fixed.push(message); | ||
| } |
There was a problem hiding this comment.
Else throw an error, logging the filename as having been ignored.
Might want to also loosen the endsWith tests - to eg filename.toLowerCase().indexOf('_add') !== -1 so stuff like 5678_Added.md will still be picked up.
There was a problem hiding this comment.
I think we have a test which disallows use of uppercase in filenames.
There was a problem hiding this comment.
We do test for uppercase, but I don't think it'll dig into the draftlogs dir. And there's still add vs adds vs added etc. Anyway the main thing is to throw an error if we ignore the file. After adding that if you still want to be strict about names that's ok, it'll just occasionally make more work during release if we get an incorrect name.
|
Looks good! Can you add notes on creating these md files to the PR template, and add notes on the new package scripts to the release procedure? |
Good call. Done in eefab9b. |
Co-authored-by: Alex Johnson <johnson.alex.c@gmail.com>
Co-authored-by: Alex Johnson <johnson.alex.c@gmail.com>
Resolves #5778.
Running
simply updates
CHANGELOG.md.@plotly/plotly_js