Skip to content
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 PRs and issues from the 1.0 milestone to the changelog #1189

Merged
merged 13 commits into from
Jun 9, 2018

Conversation

kienstra
Copy link
Contributor

@kienstra kienstra commented May 31, 2018

Request For Review

Hi @westonruter,
Could you please review this update to the Changelog?

Closes #1108.

Mainly use PRs, except where there was
extensive discussion on an issue.
Most entries don't list the corresponding
issue for a pull request.
This is in reverse-chronological order.
And it goes from the bottom up:
https://github.com/Automattic/amp-wp/milestone/7?closed=1
@kienstra kienstra added this to the v1.0 milestone Jun 1, 2018
kienstra added 3 commits May 31, 2018 19:37
Before, I had paraphrased some PR titles in the changelog.
Other edits include wrapping HTML elements in `
To improve the dispaly in Markdown.
Also, remove 'Gutenberg' from 'Gutenberg blocks.'
It sounds like that name won't be present
once it's merged into Core.
@kienstra kienstra requested a review from westonruter June 1, 2018 01:30
@westonruter
Copy link
Member

@kienstra instead of sorting in reverse chronological order, could they be generally sorted by importance? What I have in mind is something similar to the “what's new” posts for Gutenberg, for example: https://make.wordpress.org/core/2018/05/18/whats-new-in-gutenberg-18th-may/

@kienstra
Copy link
Contributor Author

kienstra commented Jun 1, 2018

Sounds good, @westonruter. I'll sort these by order of importance, like in the Gutenberg post you mentioned.

With their relative importance in mind.
Of course, this is subjective.
@kienstra
Copy link
Contributor Author

kienstra commented Jun 1, 2018

Hi @westonruter,
How does the updated Changelog look? My opinion of the relative importance is subjective 😄

@postphotos
Copy link
Contributor

Hey @kienstra, this makes my job easier and looks better than before! Thanks for this! 🎆

@westonruter westonruter merged commit 0f3491a into develop Jun 9, 2018
@westonruter westonruter deleted the update/changelog-1.0-alpha branch June 9, 2018 17:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Update changelog for v1.0 release [placeholder]
3 participants