-
Notifications
You must be signed in to change notification settings - Fork 23.1k
Add Firefox release automation script #41897
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
Conversation
8cb1954 to
060c4ed
Compare
060c4ed to
d8b6afd
Compare
d8b6afd to
a6b2c34
Compare
|
I ran: node helpers/release-firefox.js 146
# ...
modified: files/en-us/mozilla/firefox/releases/145/index.md
modified: files/en-us/mozilla/firefox/releases/146/index.md
modified: files/en-us/mozilla/firefox/releases/147/index.md
new file: files/en-us/mozilla/firefox/releases/148/index.md
Note that if you run for a past version, you will get: So I presume in the current state, if you run this on or after the day of release, it will trip on that. |
bsmth
left a comment
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.
Only thing to sort is where it lives, but once it's in a good home, this is good to merge, IMO. Do we want to couple this with a CODEOWNER change for convenience?
|
Any problem with just putting all scripts under |
That's what I initially thought. But it turned out we need to differentiate content helpers from scripts used for build and CI, as they have different levels of importance and need to be reviewed by the engineering team. |
|
You can just add another entry in the CODEOWNERS file though. Earlier entries override later ones. |
If it's in # Scripts and Tests
/scripts @mdn/engineering
+/scripts/content @mdn/content-team
/tests @mdn/engineering |
a6b2c34 to
1375115
Compare
|
@bsmth Thanks! I implemented your suggestions:
It looks ready for the upcoming release project :) |
bsmth
left a comment
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.
Looking good, thanks. Tested again with:
node scripts/content/release-firefox.js 146
And works as expected 👍🏻
|
Thanks everyone! |
Description
It’s an automation script that helps to create Firefox release notes.
I created the Firefox 145 release notes PR using this script.
Motivation
To simplify release notes workflow.
Additional details
How it works, for example, with the newly released Firefox 145:
What it does:
Takes the newly released version as a parameter (N), and:
For release dates, it uses official JSON API — https://whattrainisitnow.com/api/firefox/calendar/future/
Related issues and pull requests