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

ja: Replace {{Bug}} to markdown link #12241

Merged
merged 7 commits into from
Mar 18, 2023
Merged

Conversation

manjuu-eater
Copy link
Contributor

@manjuu-eater manjuu-eater commented Mar 17, 2023

Description

note: RegExp is \{\{\s*Bug\s*\(\s*(["']|)(\d+)\1\s*\)\s*\}\} to [Firefox bug $2](https://bugzil.la/$2)

廃止されたらしいマクロ {{Bug}} をVSCodeの置換でマークダウンのリンクに変換しました。
いちおう変なのが混ざっていないかは置換のプレビューのリストで軽く見ています。

#12028 と同じ目的のPRですが、ちょっと内容に変更があります。

大きな変更として、変換後のリンクのタイトルを
「バグ 12345」から「Firefox バグ 12345」に変更しました。(英語版に合わせました)

以下は、#12028 で言及したイレギュラーなものです。

  • files/ja/conflicting/web/api/document/index.md
    • 以前は手を付けていなかった"conflicting"以下のマクロも変換しました。
  • files/ja/mdn/writing_guidelines/page_structures/macros/commonly_used_macros/index.md
    • マクロ{{Bug}}を要求する文はen-US版ではmdリンクになって残っていたので、合わせるように変換しました。
  • files/ja/mozilla/firefox/experimental_features/index.md
    • マクロ{{Bug}}を要求する文はen-US版に合わせて章ごと削除しました。(変更なし)
  • files/ja/mozilla/firefox/releases/10/index.md
  • files/ja/mozilla/firefox/releases/68/index.md
  • files/ja/web/api/htmlmediaelement/capturestream/index.md
  • files/ja/mozilla/firefox/releases/15/index.md
    • 「(bug 480376)」はマクロではなかったですが、[Firefox バグ 480376](https://bugzil.la/480376) に変換しました。(#12028と同様)

以上です。
よろしくお願いします。

Motivation

Additional details

Related issues and pull requests

Relates to #12028

Relates to #5603
Relates to mdn/content#24834

@manjuu-eater manjuu-eater requested a review from a team as a code owner March 17, 2023 10:19
@manjuu-eater manjuu-eater requested review from kenji-yamasaki and removed request for a team March 17, 2023 10:19
@github-actions github-actions bot added the l10n-ja Issues related to Japanese content. label Mar 17, 2023
@potappo
Copy link
Collaborator

potappo commented Mar 18, 2023

元々レビューしていたものなので引き続き実施します。

@potappo
Copy link
Collaborator

potappo commented Mar 18, 2023

修正ありがとうございました。修正内容に問題ありませんので、このあとマージしますが、質問に回答しておきます。

  * 「{{bug("file-fallout")}}」は [バグ file-fallout](https://bugzil.la/file-fallout ) に変換してたのですが、en-US版に合わせて[Firefox バグ 1558299](https://bugzil.la/1558299) にしました。(元のは1566172ですけどいいんでしょうか)

より Firefox 68 に関連するバグに修正されたと思っています。レビューもされていますし、全く関係ない番号に変えたとは考えにくいです。

@potappo potappo merged commit 26d73aa into mdn:main Mar 18, 2023
@manjuu-eater manjuu-eater deleted the replace-bug-ja branch March 19, 2023 04:10
@manjuu-eater
Copy link
Contributor Author

ありがとうございました!
遅れてすみません。

丁寧なご回答助かりました。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
l10n-ja Issues related to Japanese content.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants