Skip to content
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

Deprecate {{exception}} macro #6163

Merged
merged 1 commit into from
May 4, 2022
Merged

Deprecate {{exception}} macro #6163

merged 1 commit into from
May 4, 2022

Conversation

teoli2003
Copy link
Contributor

In mdn/content#15677, I removed the last occurrences of the {{exception}} macro.

In order to be sure this macro doesn't sneak back in mdn/content while it gets removed from mdn/translated-content, we want a MacroDeprecationError flaw to be raised if a page is committed again with this macro.

This flaw can be noticed by the committer, the reviewer, and, via the flaw dashboard, all mdn/content owners. (Given we normally have 0 such flaws, it will be very noticeable)

Copy link
Contributor

@caugner caugner left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, good initiative, thank you!

@caugner caugner merged commit d774d9d into mdn:main May 4, 2022
OnkarRuikar pushed a commit to OnkarRuikar/yari that referenced this pull request Jun 2, 2022
@teoli2003 teoli2003 mentioned this pull request Aug 26, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants