Skip to content

fix(typo): React がなどが → React などが#24642

Merged
mfuji09 merged 2 commits intomdn:mainfrom
clumsy-ug:clumsy-ug-patch1
Nov 23, 2024
Merged

fix(typo): React がなどが → React などが#24642
mfuji09 merged 2 commits intomdn:mainfrom
clumsy-ug:clumsy-ug-patch1

Conversation

@clumsy-ug
Copy link
Contributor

Description

「React がなどが」を「React などが」に修正しました。

Motivation

不適切な助詞の重複使用がされているので、変更すべきだと感じました。

Additional details

なし

Related issues and pull requests

Fixes: mozilla-japan/translation#796

@clumsy-ug clumsy-ug requested a review from a team as a code owner November 21, 2024 03:34
@clumsy-ug clumsy-ug requested review from potappo and removed request for a team November 21, 2024 03:34
@github-actions github-actions bot added the l10n-ja Issues related to Japanese content. label Nov 21, 2024
@github-actions
Copy link
Contributor

github-actions bot commented Nov 21, 2024

Preview URLs

Flaws (40)

URL: /ja/docs/Learn/JavaScript/Client-side_web_APIs/Introduction
Title: Web API の紹介
Flaw count: 40

  • macros:
    • /ja/docs/Games/Publishing_games does not exist but fell back to /en-US/docs/Games/Publishing_games
    • /ja/docs/Learn/Tools_and_testing/Client-side_JavaScript_frameworks/React_interactivity_events_state does not exist but fell back to /en-US/docs/Learn/Tools_and_testing/Client-side_JavaScript_frameworks/React_interactivity_events_state
    • /ja/docs/Learn/Tools_and_testing/Client-side_JavaScript_frameworks/React_interactivity_filtering_conditional_rendering does not exist but fell back to /en-US/docs/Learn/Tools_and_testing/Client-side_JavaScript_frameworks/React_interactivity_filtering_conditional_rendering
    • /ja/docs/Learn/Tools_and_testing/Client-side_JavaScript_frameworks/React_accessibility does not exist but fell back to /en-US/docs/Learn/Tools_and_testing/Client-side_JavaScript_frameworks/React_accessibility
    • /ja/docs/Learn/Tools_and_testing/Client-side_JavaScript_frameworks/Ember_getting_started does not exist but fell back to /en-US/docs/Learn/Tools_and_testing/Client-side_JavaScript_frameworks/Ember_getting_started
    • and 34 more flaws omitted
  • broken_links:
    • Can't resolve /ja/docs/Web/Guide/Audio_and_video_delivery

(comment last updated: 2024-11-21 10:31:51)

Copy link
Collaborator

@mfuji09 mfuji09 left a comment

Choose a reason for hiding this comment

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

1件の確認をお願いします。

…dex.md

Co-authored-by: Masahiro FUJIMOTO <mfujimot@gmail.com>
@clumsy-ug
Copy link
Contributor Author

clumsy-ug commented Nov 21, 2024

申し訳ございません、誤解しておりました。
sourceCommitを変更前の状態に戻すUpdateを行いました。いかがでしょうか。

@clumsy-ug
Copy link
Contributor Author

clumsy-ug commented Nov 21, 2024

https://mozilla-japan.github.io/mdn-translation-guide/translation/6_edit-ja-file.html
また、こちらのurlにて、

軽微な修正のみを行いたい場合で、l10n が記述されていない場合には上記の Front-matter の編集はしなくても構いません。

という指示がありますが、私の場合は軽微な修正ではあったものの l10n はすでに記述されていたので l10n のsourceCommitを上書きする必要があるのだろうと考えてしまいました。

「軽微な修正の場合は l10n とすでに記述されていても記述されていなくても上書きする必要は無い」という認識で良いでしょうか?

その場合、

l10n が記述されていない場合には

の部分は無くして、

軽微な修正のみを行いたい場合は上記の Front-matter の編集はしないでください。

と変更した方が良いのではないかと思ったのですが、いかがでしょうか。

(細かい話で恐縮です。お節介でしたらスルーしてください)

@mfuji09
Copy link
Collaborator

mfuji09 commented Nov 23, 2024

@clumsy-ug さん
ご意見ありがとうございます。

sourceCommit: の更新を行うのは、PRによる修正が軽微かどうかにかかわらず、すでに英語版での最新版の翻訳を反映済みであることを示すためです。
MDN で提供されているツールを使用すると、現在の翻訳(sourceCommit: を設定した版)が英語版の最新版と比較してどれだけ古いか(または最新なのか)を確認することができます。

つまり、sourceCommit: の更新(追加)は、修正の大小にかかわらず、英語版での更新をすべて反映したことを確認したときに行ってください。

現在の日本語版の内容が最新の英語版と比較して問題ないのであれば(英語版のほうで誤字が修正されただけの場合など)、本文を1文字も変更せずに sourceCommit: のみを更新する場合もあります。

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

Copy link
Collaborator

@mfuji09 mfuji09 left a comment

Choose a reason for hiding this comment

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

ありがとうございます。OKです。

@mfuji09 mfuji09 merged commit b1068a8 into mdn:main Nov 23, 2024
@clumsy-ug clumsy-ug deleted the clumsy-ug-patch1 branch February 28, 2025 20:36
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.

リクエスト(Typo): JavaScript と API とその他 JavaScript ツールの関係 の更新

2 participants