-
Notifications
You must be signed in to change notification settings - Fork 26
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
Bug 1891921 - Use mozilla-version to determine the category o… #1429
base: main
Are you sure you want to change the base?
Conversation
dae192e
to
e56d53d
Compare
e56d53d
to
727f44f
Compare
727f44f
to
a9b27c1
Compare
See what the next commit in product details will look like: https://gist.github.com/JohanLorenzo/744122b093f6d5edfbe46488f91aedc0. There are several type of changes:
|
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'm OK with this. Please have someone in RelMan review the expected changes to product details before landing though.
To be honest, I'm not sure how to even evaluate what the downstream implications of these changes are. I would definitely like @pascalchevrel to take a look when he's back next week as I suspect he may have some better knowledge about who the downstream consumers are of these json files. As a I mentioned to jlorenzo already, I'd definitely like to confirm that this won't cause the ESR point releases to disappear from https://www.mozilla.org/firefox/releases/ to start. |
in the patch, in - "125.0.2": "2024-04-22"
+ "125.0.2": "2024-04-16" the initial date of "2024-04-22" was correct Apart from that, I don't see changes that would obviously break stuff that I know of. I tested the patched version of the files with whattrainisnitnow.com which makes heavy use of product-details data and it works. LGTM |
Status update: I plan to update my patch to avoid making too many changes to product-details. I chatted with the Bedrock team and it's better if we keep the data the way it is. Otherwise, we will break https://www.mozilla.org/en-US/firefox/releases/, for instance. I'll get back to this PR when I'm back from vacation. |
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.
Requesting changes based on the April 30 comment.
…f a release
Depends on mozilla-releng/mozilla-version#109 mozilla-releng/mozilla-version#110 and mozilla-releng/mozilla-version#111
I'm confident the current implementation is good. See what staging produced in product-details: mozilla-releng/product-details@fa9d823. See also mozilla-releng/mozilla-version#111 (comment) about expected changes in prod.