-
Notifications
You must be signed in to change notification settings - Fork 328
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
Add changelog for v3.6.0 (unreleased) release #1756
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.
LGTM. @m-green would you be okay to add a note about https://github.com/alphagov/govuk-frontend/pull/1711/files#r387609856? 🙏
@hannalaakso Added the note about |
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 👍 Just left a couple of small comments.
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.
Thanks Hanna! Fixes / comments below...
Looks great @m-green 👍 |
|
||
[Pull request #1723: Add display override for hiding content when printing](https://github.com/alphagov/govuk-frontend/pull/1723). | ||
|
||
#### The `iff` Sass function is now deprecated |
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.
Very much non-blocking, thinking out loud… should we consider splitting deprecations out into their own section? This isn't really a new feature.
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.
Might be an idea for a future version? I'm not sure about giving them a whole section to themselves, but maybe a subsection somewhere...
This adds the full edited version of the v3.6.0 release note in CHANGELOG.md.
Please review for:
Following review and before publication we need to: