Skip to content

Make all JSON code syntactically valid #41120

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
Merged
merged 2 commits into from
Sep 14, 2025
Merged

Make all JSON code syntactically valid #41120

merged 2 commits into from
Sep 14, 2025

Conversation

Josh-Cena
Copy link
Member

This makes the code formattable.

@Josh-Cena Josh-Cena requested review from a team as code owners September 14, 2025 02:17
@Josh-Cena Josh-Cena requested review from sideshowbarker and willdurand and removed request for a team, sideshowbarker and willdurand September 14, 2025 02:17
@github-actions github-actions bot added Content:WebExt WebExtensions docs Content:WebAPI Web API docs Content:Learn Learning area docs Content:WebDriver WebDriver docs Content:Meta Content in the meta docs Content:PWA Progressive Web Apps content labels Sep 14, 2025
@Josh-Cena Josh-Cena requested a review from bsmth September 14, 2025 02:17
@github-actions github-actions bot added the size/l [PR only] 501-1000 LoC changed label Sep 14, 2025
@github-actions github-actions bot added size/m [PR only] 51-500 LoC changed and removed Content:WebExt WebExtensions docs size/l [PR only] 501-1000 LoC changed labels Sep 14, 2025
@github-actions GitHub Actions
Copy link
Contributor

github-actions bot commented Sep 14, 2025 •
edited
Loading

Preview URLs (26 pages)
Flaws (4)

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

URL: /en-US/docs/Web/WebDriver/Reference/Capabilities/firefoxOptions
Title: firefoxOptions
Flaw count: 2

  • broken_links:
    • Can't resolve /en-US/docs/Web/WebDriver/Reference/Commands/NewSession
    • Can't resolve /en-US/docs/Web/WebDriver/Reference/Commands/NewSession

URL: /en-US/docs/Web/WebDriver/Reference/Capabilities/webSocketUrl
Title: webSocketUrl
Flaw count: 2

  • broken_links:
    • Can't resolve /en-US/docs/Web/WebDriver/Reference/Commands/NewSession
    • Can't resolve /en-US/docs/Web/WebDriver/Reference/Commands/NewSession

(comment last updated: 2025-09-14 23:40:19)

hamishwillee
hamishwillee approved these changes Sep 14, 2025
View reviewed changes
Copy link
Collaborator

@hamishwillee hamishwillee left a comment •
edited
Loading

Choose a reason for hiding this comment

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

Makes sense to me - thank you. Approving. Merge at will.

@Josh-Cena Josh-Cena merged commit 6d36361 into mdn:main Sep 14, 2025
8 checks passed
@Josh-Cena Josh-Cena deleted the json-syntax branch September 15, 2025 00:16
wbamberg added a commit to wbamberg/content that referenced this pull request Sep 15, 2025
@wbamberg
Merge remote-tracking branch 'upstream/main' into attacks-phishing
18cfefb 
* upstream/main: (32 commits)
  FF143 GPU import external texture experimental relnote (mdn#41126)
  added ff release note for nested pseudo-elements (mdn#41146)
  Add a page for SSRF attacks (mdn#41105)
  New CSSOM view API landing page, move CSS guides under (mdn#40547)
  40778 color input value (mdn#41086)
  Synchronize with BCD v7.1.5 (mdn#41116)
  Cross-reference `:target` and `::target-text` pages (mdn#41121)
  Fix content issues (mdn#41125)
  Make all JSON code syntactically valid (mdn#41120)
  Fix the link name with actual heading (mdn#41117)
  Associated listed form controls (mdn#41096)
  Mention `new Set()` accepts `undefined` (mdn#41106)
  chore: correct the `timeRange` function name (mdn#41107)
  Mention that custom elements constructors can't be directly called (mdn#41069)
  chore(deps): bump @mdn/fred from 1.3.2 to 1.4.0 (mdn#41111)
  Public-key cryptography glossary page: add headings and links (mdn#41092)
  Improve caret color demo contrast (mdn#41104)
  Fix supported algorithms for HKDF and PBKDF2 (mdn#41100)
  Update MongoDB example to async/await (driver v5+) (mdn#41102)
  Remove onload and DOMContentLoaded handlers (mdn#41079)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@hamishwillee hamishwillee hamishwillee approved these changes

@bsmth bsmth Awaiting requested review from bsmth

Assignees
No one assigned
Labels
Content:Learn Learning area docs Content:Meta Content in the meta docs Content:PWA Progressive Web Apps content Content:WebAPI Web API docs Content:WebDriver WebDriver docs size/m [PR only] 51-500 LoC changed
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@Josh-Cena @hamishwillee