-
Notifications
You must be signed in to change notification settings - Fork 22.5k
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
Version number simplification for MV3 #21699
Conversation
Preview URLs
External URLs (2)URL:
(this comment was updated 2022-10-31 22:48:09.039885) |
files/en-us/mozilla/add-ons/webextensions/manifest.json/version/format/index.md
Outdated
Show resolved
Hide resolved
files/en-us/mozilla/add-ons/webextensions/manifest.json/version/index.md
Outdated
Show resolved
Hide resolved
files/en-us/mozilla/add-ons/webextensions/manifest.json/version/format/index.md
Show resolved
Hide resolved
files/en-us/mozilla/add-ons/webextensions/manifest.json/version/format/index.md
Outdated
Show resolved
Hide resolved
files/en-us/mozilla/add-ons/webextensions/manifest.json/version/format/index.md
Outdated
Show resolved
Hide resolved
files/en-us/mozilla/add-ons/webextensions/manifest.json/version/index.md
Outdated
Show resolved
Hide resolved
files/en-us/mozilla/add-ons/webextensions/manifest.json/version/format/index.md
Outdated
Show resolved
Hide resolved
files/en-us/mozilla/add-ons/webextensions/manifest.json/version/format/index.md
Outdated
Show resolved
Hide resolved
files/en-us/mozilla/add-ons/webextensions/manifest.json/version/format/index.md
Outdated
Show resolved
Hide resolved
files/en-us/mozilla/add-ons/webextensions/manifest.json/version/format/index.md
Outdated
Show resolved
Hide resolved
files/en-us/mozilla/add-ons/webextensions/manifest.json/version/format/index.md
Outdated
Show resolved
Hide resolved
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 really like the direction of this PR. The readability and level of information has improved significantly.
files/en-us/mozilla/add-ons/webextensions/manifest.json/version/format/index.md
Show resolved
Hide resolved
files/en-us/mozilla/add-ons/webextensions/manifest.json/version/index.md
Outdated
Show resolved
Hide resolved
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.
Once Rob's comments have been addressed, r+wc from my side
files/en-us/mozilla/add-ons/webextensions/manifest.json/version/index.md
Outdated
Show resolved
Hide resolved
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.
The last update didn't cover #21699 (comment). Was that intentional or pending clarification?
Other than that and the comments here, it's basically ready to merge.
files/en-us/mozilla/add-ons/webextensions/manifest.json/version/format/index.md
Show resolved
Hide resolved
files/en-us/mozilla/add-ons/webextensions/manifest.json/version/index.md
Outdated
Show resolved
Hide resolved
@Rob--W I believe I've addressed all your feedback. I'll merge tomorrow if you have no further feedback. |
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!
Description
Updates the information about version number format to provide for the simplified format for MV3 extensions in Firefox 108 and later.
Related issues and pull requests
Addresses the documentation needs of Bug 1793925 Warn about complex versioning formats used in manifest