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

chore(ja): content clean up for markdown-lint-cli bump #17239

Merged
merged 3 commits into from
Dec 9, 2023
Merged

chore(ja): content clean up for markdown-lint-cli bump #17239

merged 3 commits into from
Dec 9, 2023

Conversation

yin1999
Copy link
Member

@yin1999 yin1999 commented Dec 3, 2023 •
edited
Loading

Description

content clean up for markdown-lint-cli bump

Since I can't speak Japanese, so feel free to correct any mistake in this PR.

Related issues and pull requests

prepare for: #17042
config ref: #15106

@yin1999 yin1999 requested a review from a team as a code owner December 3, 2023 05:53
@yin1999 yin1999 requested review from potappo and removed request for a team December 3, 2023 05:53
@github-actions github-actions bot added the l10n-ja Issues related to Japanese content. label Dec 3, 2023
@github-actions GitHub Actions
Copy link
Contributor

github-actions bot commented Dec 3, 2023 •
edited
Loading

Preview URLs (27 pages)
Flaws (280)

Note! 7 documents with no flaws that don't need to be listed. 🎉

URL: /ja/docs/MDN/At_ten/History_of_MDN
Title: MDN のあゆみ
Flaw count: 4

  • macros:
    • The fourth and fifth parameters of 'EmbedLiveSample' are deprecated
  • broken_links:
    • Can't resolve /ja/docs/MDN/Tools/KumaScript
    • Can't resolve /ja/docs/MDN/Contribute/Localize
  • translation_differences:
    • Differences in the important macros (0 in common of 3 possible)

URL: /ja/docs/Learn/Tools_and_testing/Understanding_client-side_tools/Command_line
Title: コマンドライン短期集中講座
Flaw count: 51

  • macros:
    • /ja/docs/Learn/MathML/First_steps does not exist but fell back to /en-US/docs/Learn/MathML/First_steps
    • /ja/docs/Learn/MathML/First_steps/Getting_started does not exist but fell back to /en-US/docs/Learn/MathML/First_steps/Getting_started
    • /ja/docs/Learn/MathML/First_steps/Text_containers does not exist but fell back to /en-US/docs/Learn/MathML/First_steps/Text_containers
    • /ja/docs/Learn/MathML/First_steps/Fractions_and_roots does not exist but fell back to /en-US/docs/Learn/MathML/First_steps/Fractions_and_roots
    • /ja/docs/Learn/MathML/First_steps/Scripts does not exist but fell back to /en-US/docs/Learn/MathML/First_steps/Scripts
    • and 46 more flaws omitted

URL: /ja/docs/Learn/Tools_and_testing/Cross_browser_testing/Introduction
Title: はじめてのクロスブラウザーテスト
Flaw count: 54

  • macros:
    • /ja/docs/Learn/MathML/First_steps does not exist but fell back to /en-US/docs/Learn/MathML/First_steps
    • /ja/docs/Learn/MathML/First_steps/Getting_started does not exist but fell back to /en-US/docs/Learn/MathML/First_steps/Getting_started
    • /ja/docs/Learn/MathML/First_steps/Text_containers does not exist but fell back to /en-US/docs/Learn/MathML/First_steps/Text_containers
    • /ja/docs/Learn/MathML/First_steps/Fractions_and_roots does not exist but fell back to /en-US/docs/Learn/MathML/First_steps/Fractions_and_roots
    • /ja/docs/Learn/MathML/First_steps/Scripts does not exist but fell back to /en-US/docs/Learn/MathML/First_steps/Scripts
    • and 46 more flaws omitted
  • broken_links:
    • Can't resolve /ja/docs/Learn/CSS/Introduction_to_CSS/Debugging_CSS
    • Is currently http:// but can become https://
    • Can't resolve /ja/docs/Learn/CSS/Introduction_to_CSS/Debugging_CSS

URL: /ja/docs/Mozilla/Firefox/Releases/16
Title: Firefox 16 for developers
Flaw count: 16

  • macros:
    • /ja/docs/Web/CSS/-moz-appearance redirects to /ja/docs/Web/CSS/appearance
    • /ja/docs/Web/API/Keyboard/setSelectedOption does not exist
    • /ja/docs/Web/API/Keyboard/setValue does not exist
    • /ja/docs/Web/API/Keyboard/onfocuschange does not exist
    • /ja/docs/Web/API/SmsRequest does not exist
    • and 2 more flaws omitted
  • broken_links:
    • Can't resolve /ja/docs/CSS/Using_CSS_Animations
    • Can't resolve /ja/docs/CSS/Using_CSS_transforms
    • Can't resolve /ja/docs/CSS/Using_CSS_gradients
    • Can't resolve /ja/docs/CSS/Media_queries
    • Can't resolve /ja/docs/DOM/window.navigator.battery
    • and 4 more flaws omitted

URL: /ja/docs/Mozilla/Firefox/Releases/64
Title: Firefox 64 for developers
Flaw count: 30

  • macros:
    • /ja/docs/Web/CSS/::-moz-tree does not exist
    • wrong xref macro used (consider changing which macro you use)
    • /ja/docs/Web/API/Fullscreenchange does not exist
    • /ja/docs/Web/API/Fullscreenerror does not exist
    • /ja/docs/Web/API/Window/screenLeft does not exist but fell back to /en-US/docs/Web/API/Window/screenLeft
    • and 10 more flaws omitted
  • broken_links:
    • Can't resolve /ja/docs/Tools/Accessibility_inspector#Highlighting_of_UI_items
    • Can't resolve /ja/docs/Tools/Responsive_Design_Mode
    • Can't resolve /ja/docs/Tools/Network_Monitor
    • Can use the English (en-US) link as a fallback
    • Can't resolve /ja/docs/Tools/Web_Console
    • and 10 more flaws omitted

URL: /ja/docs/Mozilla/Firefox/Releases/65
Title: Firefox 65 for developers
Flaw count: 11

  • macros:
    • /ja/docs/Web/JavaScript/Reference/Global_Objects/RelativeTimeFormat does not exist but fell back to /en-US/docs/Web/JavaScript/Reference/Global_Objects/Intl/RelativeTimeFormat
    • /ja/docs/Web/API/SVGImageElement does not exist but fell back to /en-US/docs/Web/API/SVGImageElement
    • /ja/docs/Web/API/SharedWorkerGlobalScope/onconnect does not exist but fell back to /en-US/docs/Web/API/SharedWorkerGlobalScope/connect_event
    • /ja/docs/Web/API/Response/redirect redirects to /ja/docs/Web/API/Response/redirect_static
    • /ja/docs/Web/API/ServiceWorkerGlobalScope/onmessageerror does not exist
    • and 4 more flaws omitted
  • broken_links:
    • Can't resolve /ja/docs/Glossary/webp
    • Can use the English (en-US) link as a fallback

URL: /ja/docs/Web/SVG/Element/defs
Title: defs
Flaw count: 14

  • macros:
    • /ja/docs/Web/SVG/Element/color-profile does not exist
    • /ja/docs/Web/SVG/Element/cursor does not exist but fell back to /en-US/docs/Web/SVG/Element/cursor
    • /ja/docs/Web/SVG/Element/filter does not exist but fell back to /en-US/docs/Web/SVG/Element/filter
    • /ja/docs/Web/SVG/Element/font does not exist but fell back to /en-US/docs/Web/SVG/Element/font
    • /ja/docs/Web/SVG/Element/font-face does not exist but fell back to /en-US/docs/Web/SVG/Element/font-face
    • and 3 more flaws omitted
  • broken_links:
    • Can't resolve /ja/SVG/Attribute#ConditionalProccessing
    • Can't resolve /ja/SVG/Attribute#Core
    • Can't resolve /ja/SVG/Attribute#GraphicalEvent
    • Can't resolve /ja/SVG/Attribute#Presentation
    • Can't resolve /ja/DOM/SVGDefsElement
  • translation_differences:
    • Differences in the important macros (3 in common of 4 possible)

URL: /ja/docs/Web/SVG/Element/animateTransform
Title: animateTransform
Flaw count: 10

  • broken_links:
    • Anchor not lowercase
    • Anchor not lowercase
    • Anchor not lowercase
    • Anchor not lowercase
    • Anchor not lowercase
    • and 4 more flaws omitted
  • translation_differences:
    • Differences in the important macros (3 in common of 5 possible)

URL: /ja/docs/Web/SVG/Element/desc
Title: desc
Flaw count: 3

  • macros:
    • /ja/docs/Web/API/SVGDescElement does not exist but fell back to /en-US/docs/Web/API/SVGDescElement
  • broken_links:
    • Anchor not lowercase
  • translation_differences:
    • Differences in the important macros (3 in common of 4 possible)

URL: /ja/docs/Web/SVG/Attribute/stroke-linecap
Title: stroke-linecap
Flaw count: 4

  • broken_links:
    • Is currently http:// but can become https://
    • Anchor not lowercase
    • Anchor not lowercase
  • translation_differences:
    • Differences in the important macros (0 in common of 8 possible)

URL: /ja/docs/Web/SVG/Attribute/stroke-linejoin
Title: stroke-linejoin
Flaw count: 4

  • broken_links:
    • Is currently http:// but can become https://
    • Can't resolve /ja/SVG/Element#Shape
    • Can't resolve /ja/SVG/Element#TextContent
  • translation_differences:
    • Differences in the important macros (0 in common of 10 possible)

URL: /ja/docs/Web/Media/DASH_Adaptive_Streaming_for_HTML_5_Video
Title: HTML 5 ビデオ用の DASH アダプティブストリーミング
Flaw count: 2

  • broken_links:
    • Is currently http:// but can become https://
    • Is currently http:// but can become https://

URL: /ja/docs/Web/Progressive_web_apps
Title: プログレッシブウェブアプリ (PWA)
Flaw count: 11

  • broken_links:
    • Can't resolve /ja/docs/Web/Progressive_web_apps/Introduction#advantages_of_web_applications
    • Can't resolve /ja/docs/Web/Progressive_web_apps/Developer_guide
    • Can't resolve /ja/docs/Web/Progressive_web_apps/Introduction
    • Can't resolve /ja/docs/Web/Progressive_web_apps/App_structure
    • Can't resolve /ja/docs/Web/Progressive_web_apps/Offline_Service_workers
    • and 6 more flaws omitted

URL: /ja/docs/Web/JavaScript/Reference/Strict_mode
Title: 厳格モード
Flaw count: 2

  • broken_links:
    • Can't resolve /ja/docs/Web/JavaScript/Reference/Errors/Strict_Non_Simple_Params
  • translation_differences:
    • Differences in the important macros (1 in common of 2 possible)

URL: /ja/docs/Web/JavaScript/Reference/Global_Objects/Object/create
Title: Object.create()
Flaw count: 1

  • macros:
    • wrong xref macro used (consider changing which macro you use)

URL: /ja/docs/Web/API/RTCPeerConnection/RTCPeerConnection
Title: RTCPeerConnection()
Flaw count: 12

  • macros:
    • /ja/docs/Web/API/RTCDtlsTransport does not exist but fell back to /en-US/docs/Web/API/RTCDtlsTransport
    • /ja/docs/Web/API/RTCDtlsTransport does not exist but fell back to /en-US/docs/Web/API/RTCDtlsTransport
    • /ja/docs/Web/API/RTCDtlsTransport does not exist but fell back to /en-US/docs/Web/API/RTCDtlsTransport
    • /ja/docs/Web/API/RTCCertificate does not exist but fell back to /en-US/docs/Web/API/RTCCertificate
    • /ja/docs/Web/API/RTCPeerConnection/setConfiguration does not exist but fell back to /en-US/docs/Web/API/RTCPeerConnection/setConfiguration
    • and 6 more flaws omitted
  • broken_links:
    • Can use the English (en-US) link as a fallback

URL: /ja/docs/Web/API/SVGElement
Title: SVGElement
Flaw count: 24

  • macros:
    • /ja/docs/Web/API/HTMLOrForeignElement redirects to /ja/docs/Web/API/HTMLElement
    • /ja/docs/Web/API/SVGElement/dataset does not exist
    • /ja/docs/Web/API/SVGElement/className does not exist
    • /ja/docs/Web/API/SVGAnimatedString does not exist but fell back to /en-US/docs/Web/API/SVGAnimatedString
    • /ja/docs/Web/API/SVGElement/ownerSVGElement does not exist
    • and 5 more flaws omitted
  • broken_links:
    • Can use the English (en-US) link as a fallback
    • Can't resolve /ja/docs/Web/API/SVGElement/abort_event
    • Can't resolve /ja/docs/Web/API/GlobalEventHandlers/onabort
    • Can use the English (en-US) link as a fallback
    • Can use the English (en-US) link as a fallback
    • and 9 more flaws omitted

URL: /ja/docs/Web/API/Web_Authentication_API
Title: ウェブ認証 API
Flaw count: 17

  • macros:
    • /ja/docs/Web/API/CredentialsContainer/get does not exist but fell back to /en-US/docs/Web/API/CredentialsContainer/get
    • /ja/docs/Web/API/PublicKeyCredentialCreationOptions redirects to /ja/docs/Web/API/CredentialsContainer/create
    • /ja/docs/Web/API/AuthenticatorResponse/clientDataJSON does not exist but fell back to /en-US/docs/Web/API/AuthenticatorResponse/clientDataJSON
    • /ja/docs/Web/API/AuthenticatorResponse/clientDataJSON does not exist but fell back to /en-US/docs/Web/API/AuthenticatorResponse/clientDataJSON
    • /ja/docs/Web/API/AuthenticatorAttestationResponse/attestationObject does not exist but fell back to /en-US/docs/Web/API/AuthenticatorAttestationResponse/attestationObject
    • and 10 more flaws omitted
  • broken_links:
    • Can use the English (en-US) link as a fallback
  • translation_differences:
    • Differences in the important macros (2 in common of 3 possible)

URL: /ja/docs/Web/API/XMLDocument
Title: XMLDocument
Flaw count: 4

  • macros:
    • /ja/docs/Web/API/XMLDocument/async redirects to /ja/docs/Web/API/XMLDocument
    • /ja/docs/Web/API/XMLDocument/load does not exist but fell back to /en-US/docs/Web/API/XMLDocument
    • /ja/docs/Web/API/XMLDocument/load does not exist but fell back to /en-US/docs/Web/API/XMLDocument
  • broken_links:
    • Can't resolve /ja/docs/DOM/DOM_Reference

URL: /ja/docs/Web/API/Web_Audio_API/Basic_concepts_behind_Web_Audio_API
Title: Basic concepts behind Web Audio API
Flaw count: 6

  • macros:
    • /ja/docs/Web/API/Float32Array does not exist but fell back to /en-US/docs/Web/JavaScript/Reference/Global_Objects/Float32Array
    • /ja/docs/Web/API/Uint8Array does not exist but fell back to /en-US/docs/Web/JavaScript/Reference/Global_Objects/Uint8Array
    • /ja/docs/Web/API/Float32Array does not exist but fell back to /en-US/docs/Web/JavaScript/Reference/Global_Objects/Float32Array
    • /ja/docs/Web/API/Uint8Array does not exist but fell back to /en-US/docs/Web/JavaScript/Reference/Global_Objects/Uint8Array
  • broken_links:
    • Can use the English (en-US) link as a fallback
  • translation_differences:
    • Differences in the important macros (0 in common of 1 possible)
External URLs (3)

URL: /ja/docs/Web/Media/Streaming
Title: オーディオとビデオのストリーミングガイド

URL: /ja/docs/Web/API/RTCPeerConnection/RTCPeerConnection
Title: RTCPeerConnection()

URL: /ja/docs/Web/API/Web_Audio_API/Basic_concepts_behind_Web_Audio_API
Title: Basic concepts behind Web Audio API

(comment last updated: 2023-12-09 01:39:37)

@github-actions GitHub Actions
Copy link
Contributor

github-actions bot commented Dec 5, 2023

This pull request has merge conflicts that must be resolved before it can be merged.

@github-actions github-actions bot added the merge conflicts 🚧 This pull request has merge conflicts that must be resolved. label Dec 5, 2023
@github-actions github-actions bot removed the merge conflicts 🚧 This pull request has merge conflicts that must be resolved. label Dec 5, 2023
mfuji09
mfuji09 requested changes Dec 7, 2023
View reviewed changes
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.

@yin1999
Thank you! I noticed about two changes.

files/ja/.markdownlint.jsonc Outdated Show resolved Hide resolved
files/ja/web/api/web_authentication_api/index.md Outdated
@@ -35,7 +35,7 @@ create() と get() メソッドが全体像の中でどのように位置づけ

![ウェブ認証 API のコンポーネントの登録とデータフロー図](webauthn_registration_r4.png)

_図 1 - ウェブ認証による登録手順と各アクションに関連する重要なデータの流れを示す図_
_図 1 - ウェブ認証による登録手順と各アクションに関連する重要なデータの流れを示す図。_
Copy link
Collaborator

Choose a reason for hiding this comment

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

We usually omit the last punctation for a caption of a diagram.

Suggested change
_図 1 - ウェブ認証による登録手順と各アクションに関連する重要なデータの流れを示す図。_
_図 1 - ウェブ認証による登録手順と各アクションに関連する重要なデータの流れを示す図_

Copy link
Member Author

Choose a reason for hiding this comment

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

Hey @mfuji09. This would cause the "Emphasis used instead of a heading" error with the new version of markdown lint if the whole sentence is emphasized without the tailing punctuation.

As the en-US document has removed the sentence. Is it okay if I add the ignore expression (<!-- markdownlint-disable -->) to this sentence for now (in a future update to this article, we can remove the ignore expression)

image

Copy link
Member Author

Choose a reason for hiding this comment

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

Fixed in 48e4765

Copy link
Collaborator

Choose a reason for hiding this comment

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

Thank you!

@yin1999 yin1999 requested a review from mfuji09 December 9, 2023 01:38
mfuji09
mfuji09 approved these changes Dec 9, 2023
View reviewed changes
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.

LGTM. Thank you!

@mfuji09 mfuji09 merged commit 4e1efcf into mdn:main Dec 9, 2023
10 checks passed
@yin1999 yin1999 deleted the ja-markdown-lint branch December 9, 2023 07:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@mfuji09 mfuji09 mfuji09 approved these changes

@potappo potappo Awaiting requested review from potappo potappo is a code owner automatically assigned from mdn/yari-content-ja

Assignees
No one assigned
Labels
l10n-ja Issues related to Japanese content.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@yin1999 @mfuji09