-
Notifications
You must be signed in to change notification settings - Fork 516
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(macros): remove page macro #11356
base: main
Are you sure you want to change the base?
Conversation
the page macro has been deprecated and is broken anyway
Running
on translated content shows the only uses in pt-BR. And there it's broken already (see https://developer.mozilla.org/pt-BR/docs/Web/API/File#propriedades) |
|
||
// Throw a MacroDeprecatedError flaw | ||
// Can be removed when its usage translated-content is down to 0 | ||
mdn.deprecated(); |
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.
@fiji-flo Since this macro is still referenced in pt-BR
, can we please keep a stub with mdn.deprecated()
so that localizers notice?
This pull request has merge conflicts that must be resolved before it can be merged. |
Summary
the page macro has been deprecated and is broken anyway