-
Notifications
You must be signed in to change notification settings - Fork 606
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
Unify date format for release notes in CHANGELOG.md #1990
Comments
i second this idea and think it might be worthwhile adopting the https://keepachangelog.com/en/1.0.0/ format @BC-tymurbiedukhin what are you thoughts on this, as I know you're often the one to request changelog entries. I could add the changes to #2080 if you like? |
We should align all dates to latest date format, it would be great to update previous values @rowleyaj |
@BC-tymurbiedukhin any chance I can convince you to move towards
I'm happy to raise a PR to update all the dates if you agree with this format |
The latest entry in the Changelog is 5.7.1 (07-09-2021) In future (after September) it won't be clear to a lot of people if that's 7th September or 9th July. |
I'm fine with updating date format, changes for stencil CLI have been requested, @rowleyaj should update existing changelog entries with a new PR |
Thanks for changing the format. However since 6.0.0 the new updates in |
@mike-healy I think i've tracked this down to this line in stencil-cli https://github.com/bigcommerce/stencil-cli/blob/master/lib/release/questions.js#L109. This appears to have been changed at the start of 2020 in this PR bigcommerce/stencil-cli#553
I'm not sure why this was done, perhaps @junedkazi can provide more context? |
The release titles in
CHANGELOG.md
reference the release date, but historically have used bothYYYY-MM-DD
andMM-DD-YYYY
.MM-DD
orDD-MM
prefixed formats can be ambiguous as different regions use each.I'd suggest using
YYYY-MM-DD
for all dates in this document.The text was updated successfully, but these errors were encountered: