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.

Sign up for GitHub

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

Jump to bottom

Macro removal - Meta - deprecated macros and macro arguments #5603

Closed
SphinxKnight opened this issue May 12, 2022 · 15 comments
Closed

Macro removal - Meta - deprecated macros and macro arguments #5603

SphinxKnight opened this issue May 12, 2022 · 15 comments
Labels
effort: large Task is large effort. help wanted If you know something about this, we would love your help! l10n-fr Issues related to French content. l10n-ko Issues related to Korean content. l10n-pt-br Issues related to Brazilian Portuguese l10n-ru Issues related to Russian content. p1 We will address this soon and will provide capacity from our team for it in the next few releases.
Milestone
Macros removal

Comments

@SphinxKnight
Copy link
Member

SphinxKnight commented May 12, 2022 •
edited by yin1999
Loading

Hello @mdn/localization-team-leads With mdn/yari#6252, the macros which are only used (because of legacy/history) in mdn/translated-content are marked as deprecated.

Here is a breakdown of each of them and their corresponding counts in this repo (as of e296c15). I hope this will make it easier for each locale's team to split the work and replace/update the pages accordingly.

Don't hesitate to ping me here or on Matrix (or to edit this directly if you have the corresponding rights) to update this list :)
N.B.
@SphinxKnight SphinxKnight added l10n-ja Issues related to Japanese content. l10n-fr Issues related to French content. l10n-zh Issues related to Chinese content. l10n-es Issues related to Spanish content. l10n-ru Issues related to Russian content. l10n-de (retired) Issues related to German content. l10n-ko Issues related to Korean content. l10n-pl (retired) Issues related to Polish content. l10n-pt-br Issues related to Brazilian Portuguese labels May 12, 2022
@alattalatta alattalatta mentioned this issue May 12, 2022
Merged
@SphinxKnight SphinxKnight mentioned this issue May 12, 2022
Closed
22 tasks
@schalkneethling
Copy link

Thank you so much for putting this together @SphinxKnight 🎉

N.B. There is the issue of de and pl which are inactive.

As @caugner mentioned in another pull request:

Removing usages of deprecated macros from frozen locales is absolutely appreciated. 👍
Background: We still build all locales, including frozen ones, every night. This means we can only remove deprecated macros if they are no longer used in any locale, including frozen locales.
Note: The locales that we no longer "care" about are elsewhere, e.g. retired-content and retired-archived-content.

@SphinxKnight
Copy link
Member Author

Removing usages of deprecated macros from frozen locales is absolutely appreciated. +1
Background: We still build all locales, including frozen ones, every night. This means we can only remove deprecated macros if they are no longer used in any locale, including frozen locales.

Thanks, I'll take care of that with "batch"-search/replace after fr

@alattalatta alattalatta mentioned this issue May 13, 2022
Merged
@yin1999 yin1999 mentioned this issue May 15, 2022
Merged
@SphinxKnight
Copy link
Member Author

I added #5715 to align vs what has been done over mdn/content in mdn/content#3196

@alattalatta alattalatta mentioned this issue May 18, 2022
Merged
@alattalatta
Copy link
Member

All macros listed here have been removed from ko docs.

This was referenced May 21, 2022
Merged
Merged
Merged
@cw118 cw118 removed the l10n-ko Issues related to Korean content. label May 24, 2022
This was referenced May 25, 2022
Merged
Merged
Merged
Merged
This was referenced Mar 26, 2023
Merged
Merged
Merged
This was referenced Apr 7, 2023
Merged
Merged
Merged
This was referenced Apr 16, 2023
Merged
Merged
Merged
Merged
This was referenced May 5, 2023
Closed
Merged
Closed
Merged
This was referenced May 13, 2023
Merged
Merged
@Lou8is Lou8is mentioned this issue Jun 6, 2023
Merged
This was referenced Jun 7, 2023
Merged
Merged
Lou8is added a commit to Lou8is/translated-content that referenced this issue Jun 8, 2023
@Lou8is
Merge branch 'main' into mdn#5603-removes-HTMLAttrXRef
b4dec94
cw118 added a commit to Lou8is/translated-content that referenced this issue Jun 10, 2023
@cw118
Merge branch 'main' into mdn#5603-removes-HTMLAttrDef
6471ad7
This was referenced Jun 10, 2023
Merged
Merged
Merged
Merged
@SphinxKnight
Copy link
Member Author

Closed in favor of https://github.com/orgs/mdn/discussions/263 as an issue should have an end.

@SphinxKnight SphinxKnight closed this as not planned Won't fix, can't repro, duplicate, stale Jul 31, 2023
@yin1999 yin1999 unpinned this issue Jul 31, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
effort: large Task is large effort. help wanted If you know something about this, we would love your help! l10n-fr Issues related to French content. l10n-ko Issues related to Korean content. l10n-pt-br Issues related to Brazilian Portuguese l10n-ru Issues related to Russian content. p1 We will address this soon and will provide capacity from our team for it in the next few releases.
Projects
MDN translated-content l10n-zh
Archived in project
Milestone
Macros removal
Development

No branches or pull requests