-
-
Notifications
You must be signed in to change notification settings - Fork 203
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
v12.0.0 release #515
v12.0.0 release #515
Conversation
The required changes because of #476 are:
|
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.
I had one minor suggestion for the changelog. But more importantly, it doesn't seem valid. The reference link definitions at the bottom have not been updated.
Edit: shameless plug for #511 😅 That would have caught this problem.
Co-authored-by: Mark Stacey <[email protected]>
Haha on it! |
CHANGELOG.md
Outdated
All notable changes to this project will be documented in this file. | ||
|
||
The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/), | ||
and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html). | ||
|
||
## [Unreleased] | ||
|
||
## [12.0.0] - 2021-07-07 | ||
|
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.
Where did all of these extra newlines come from 🤔 This would also fail validation.
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.
It came from prettier... i have to keep a .prettierignore file without committing it or its gets really opionated about .md files
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.
should be fixed 😓
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.
Huh, interesting. It's omitted in yarn lint
explicitly, but we don't have any ignore file committed.... maybe that's something we should consider 🤔 For anyone that runs prettier directly.
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.
I think prettier plugin for vscode respects eslintrc, so if were to somehow move that setting into the .eslintrc it should be fine. Otherwise a prettierignore file is also good.
8528930
to
708ac1b
Compare
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.
LGTM!
|
* v12.0.0 release * Update CHANGELOG.md Co-authored-by: Mark Stacey <[email protected]> * update ref links Co-authored-by: Mark Stacey <[email protected]>
* v12.0.0 release * Update CHANGELOG.md Co-authored-by: Mark Stacey <[email protected]> * update ref links Co-authored-by: Mark Stacey <[email protected]>
proposed changelog:
[12.0.0] - 2021-07-07
Added
Changed