-
Notifications
You must be signed in to change notification settings - Fork 6
feat(deprecations): deprecations page that lists x-glean-deprecations #232
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
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
63b43a4 to
941dbad
Compare
941dbad to
f35e198
Compare
|
@cursoragent review |
f35e198 to
4f48ee3
Compare
fa6ce5a to
d525eb2
Compare
d525eb2 to
f5f1fc9
Compare
david-hamilton-glean
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.
Looks great. Couple minor points of 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.
2/5 (minor preference, non-blocking)
is it feasible to add tests for this?
scripts/generate-deprecations.mjs
Outdated
| * | ||
| * @type {EndpointGroup[]} | ||
| */ | ||
| const EXTRA_DEPRECATIONS = []; |
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.
3/5 (strong opinion: non-blocking)
Can we add something in README that mentions how to add custom deprecations? Doesn't need to be anything involved.
Summary
Adds a new deprecations overview and list page to show all upcoming deprecations derived from x-glean-deprecated. Currently behind the x-glean-deprecated feature-flag.
Example: