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

chore(release): mark v6.0.0 #6800

Merged
merged 1 commit into from
Feb 2, 2021
Merged

chore(release): mark v6.0.0 #6800

merged 1 commit into from
Feb 2, 2021

Conversation

jschfflr
Copy link
Contributor

@jschfflr jschfflr commented Feb 2, 2021

No description provided.


### ⚠ BREAKING CHANGES

* **chromium:** The built-in `aria/` selector query handler doesn’t return ignored elements anymore.
Copy link
Member

Choose a reason for hiding this comment

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

FYI @jackfranklin, re: our discussion from yesterday, in the future let's use the suggested Conventional Commits format to mark breaking changes:

BREAKING CHANGE: Foo bar baz…

That way it automatically appears in the changelog, like this line right here. I’ll try to keep an eye out for this during reviews as well, this is all new for me too :)

@jschfflr jschfflr enabled auto-merge (squash) February 2, 2021 10:30
@mathiasbynens mathiasbynens merged commit ef47670 into main Feb 2, 2021
@mathiasbynens mathiasbynens deleted the v-6-0-0 branch February 2, 2021 11:00
@@ -17,7 +17,7 @@
const versionsPerRelease = new Map([
// This is a mapping from Chromium version => Puppeteer version.
// In Chromium roll patches, use 'NEXT' for the Puppeteer version.
['89.0.4389.0', 'NEXT'],
['89.0.4389.0', '6.0.0'],
Copy link
Member

Choose a reason for hiding this comment

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

In hindsight, this should have been 'v6.0.0' (the v is missing)

Copy link
Member

Choose a reason for hiding this comment

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

Fix: #6803

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

Successfully merging this pull request may close these issues.

None yet

2 participants