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

misc: update changelog for v6.0.0 #10807

Merged
merged 2 commits into from
May 19, 2020
Merged

misc: update changelog for v6.0.0 #10807

merged 2 commits into from
May 19, 2020

Conversation

brendankenny
Copy link
Member

way better than #10804

@brendankenny brendankenny requested a review from a team as a code owner May 19, 2020 04:44
@brendankenny brendankenny requested review from patrickhulce and removed request for a team May 19, 2020 04:44
@brendankenny brendankenny requested a review from paulirish May 19, 2020 04:44
@paulirish
Copy link
Member

Copy link
Collaborator

@patrickhulce patrickhulce left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🎉 great job consolidating everything! reads like a book :)

changelog.md Outdated
* A PWA [`maskable-icon`](https://web.dev/maskable-icon-audit/) just looks better on your homescreen, so this new audit encourages you to have at least one available in your manifest ([#10370](https://github.com/GoogleChrome/lighthouse/pull/10370)).
* `timing-budget` expands budget assertions to now be settable on all the performance metrics ([#9901](https://github.com/GoogleChrome/lighthouse/pull/9901), [#9925](https://github.com/GoogleChrome/lighthouse/pull/9925)).
* The new [`charset`](https://web.dev/charset) audit ensures a proper character encoding for page content ([#10284](https://github.com/GoogleChrome/lighthouse/pull/10284), [#10389](https://github.com/GoogleChrome/lighthouse/pull/10389), [#10689](https://github.com/GoogleChrome/lighthouse/pull/10689)).
* `image-size-responsive` checks that images have an aspect ratio and resolution that match well with how they are displaed on a page ([#10460](https://github.com/GoogleChrome/lighthouse/pull/10460)).
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
* `image-size-responsive` checks that images have an aspect ratio and resolution that match well with how they are displaed on a page ([#10460](https://github.com/GoogleChrome/lighthouse/pull/10460)).
* `image-size-responsive` checks that images have an aspect ratio and resolution that match well with how they are displayed on a page ([#10460](https://github.com/GoogleChrome/lighthouse/pull/10460)).

paulirish pushed a commit that referenced this pull request May 19, 2020
@paulirish paulirish merged commit e2bfa14 into master May 19, 2020
@paulirish paulirish deleted the changelog branch May 19, 2020 15:29
@jayaddison
Copy link
Contributor

Congratulations on the 6.0.0 release! 🎉

Selfishly-reporting: might there be an issue with the collection of contributor handles? (perhaps counting (contributions.length == 1) rather than than (contributions.since_prev_release > 0), or something like that?)

@patrickhulce
Copy link
Collaborator

Oh no! @jayaddison I'm not sure how you got cutoff I'm so sorry! 😢

This is the script that generates the contibutors

git --no-pager shortlog -s -e -n "v2.3.0..v${OLD_VERSION}" | cut -f 2 | sort > auto_contribs_prior_to_last
git --no-pager shortlog -s -e -n "v${OLD_VERSION}..HEAD" | cut -f 2 | sort > auto_contribs_since_last
NEW_CONTRIBUTORS=$(comm -13 auto_contribs_prior_to_last auto_contribs_since_last)

When I run it with 5.6.0 as the OLD_VERSION I get you in the list, so I'm not sure what happened. Perhaps there was something squirrely with the fact that we published a 6.0.0-beta in between? 😕

Output of Above Command with Emails Redacted

Alex Tkachuk
AndreasKubasa
Ben Schwarz
Carr.z.w
Catalin Rosu
Elizabeth Sweeny
François Beaufort
James Addison
James Addison
Jan Nicklas
John Sampson
Kiko Beats
Maisam Somji
Markus Staab
Mathias Bynens
Matt Jared
Michael Blasingame
Michael Dijkstra
Peter Müller
Piotr Zarycki
Robert Linder
Roelf-Jan de Vries
Roland Burrows
Sebastian Kreft
SphinxKnight
Tim Giles Jr
Umar Hansa
Warren Maresca
Warren Maresca
Web Design Hertfordshire
baseeee
jazyan
jazyan
kmanuel
nxqamar
ruxin
⭐ B3none

@jayaddison
Copy link
Contributor

No worries! I make that 34 unique contributors, and the changelog mentions 29 (please check my math), so there may have been a few folks left out accidentally.

@brendankenny
Copy link
Member Author

Arg, really sorry @jayaddison! That's my fault. Something got lost mapping from shortlog -> commit -> github username.

No worries! I make that 34 unique contributors, and the changelog mentions 29 (please check my math), so there may have been a few folks left out accidentally.

I lost one other, but there are also three old-time committers in that list (benschwarz (#2355), beaufortfrancois (#1032), and umaar (all the way back to #150!)).

@jayaddison
Copy link
Contributor

All good, thank you for the swift fix :)

@connorjclark
Copy link
Collaborator

I lost one other, but there are also three old-time committers in that list (benschwarz (#2355), beaufortfrancois (#1032), and umaar (all the way back to #150!)).

maybe emails changed? if so, perhaps adding them to .mailmap will prevent.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants