Skip to content

Fix malformed arrays#4808

Merged
kenyonj merged 24 commits intomainfrom
kenyonj-patch-1
Jun 25, 2025
Merged

Fix malformed arrays#4808
kenyonj merged 24 commits intomainfrom
kenyonj-patch-1

Conversation

@kenyonj
Copy link
Contributor

@kenyonj kenyonj commented Jun 25, 2025

Please confirm this pull request meets the following requirements:

Which change are you proposing?

  • Suggesting edits to an existing topic or collection
  • Curating a new topic or collection
  • Something that does not neatly fit into the binary options above

Editing an existing topic or collection

I'm suggesting these edits to an existing topic or collection:

  • Image (and my file is *.png, square, dimensions 288x288, size <= 75 kB)
  • Content (and my changes are in index.md)

Please replace this line with an explanation of why you think these changes should be made.

Curating a new topic or collection

  • I've formatted my changes as a new folder directory, named for the topic or collection as it appears in the URL on GitHub (e.g. https://github.com/topics/[NAME] or https://github.com/collections/[NAME])
  • My folder contains a *.png image (if applicable) and index.md
  • All required fields in my index.md conform to the Style Guide and API docs: https://github.com/github/explore/tree/main/docs

Please replace this line with an explanation of why you think this topic or collection should be curated.

Something that does not neatly fit into the binary options above

  • My suggested edits are not about an existing topic or collection, or at least not a single one
  • My suggested edits are not about curating a new topic or collection, or at least not a single one
  • My suggested edits conform to the Style Guide and API docs: https://github.com/github/explore/tree/main/docs

Please replace this line with an explanation of your proposed changes.


Please note: we will close your PR without comment if you do not check the boxes above and provide ALL requested information.

kenyonj added 24 commits June 25, 2025 18:37
Signed-off-by: Justin Kenyon <kenyonj@github.com>
Signed-off-by: Justin Kenyon <kenyonj@github.com>
Signed-off-by: Justin Kenyon <kenyonj@github.com>
Signed-off-by: Justin Kenyon <kenyonj@github.com>
Signed-off-by: Justin Kenyon <kenyonj@github.com>
Signed-off-by: Justin Kenyon <kenyonj@github.com>
Signed-off-by: Justin Kenyon <kenyonj@github.com>
Signed-off-by: Justin Kenyon <kenyonj@github.com>
Signed-off-by: Justin Kenyon <kenyonj@github.com>
Signed-off-by: Justin Kenyon <kenyonj@github.com>
Signed-off-by: Justin Kenyon <kenyonj@github.com>
Signed-off-by: Justin Kenyon <kenyonj@github.com>
Signed-off-by: Justin Kenyon <kenyonj@github.com>
Signed-off-by: Justin Kenyon <kenyonj@github.com>
Signed-off-by: Justin Kenyon <kenyonj@github.com>
Signed-off-by: Justin Kenyon <kenyonj@github.com>
Signed-off-by: Justin Kenyon <kenyonj@github.com>
Signed-off-by: Justin Kenyon <kenyonj@github.com>
Signed-off-by: Justin Kenyon <kenyonj@github.com>
Signed-off-by: Justin Kenyon <kenyonj@github.com>
Signed-off-by: Justin Kenyon <kenyonj@github.com>
Signed-off-by: Justin Kenyon <kenyonj@github.com>
Signed-off-by: Justin Kenyon <kenyonj@github.com>
Signed-off-by: Justin Kenyon <kenyonj@github.com>
@kenyonj kenyonj marked this pull request as ready for review June 25, 2025 22:48
Copilot AI review requested due to automatic review settings June 25, 2025 22:48
@kenyonj kenyonj requested a review from a team as a code owner June 25, 2025 22:48
@kenyonj kenyonj merged commit 5eaabd1 into main Jun 25, 2025
7 of 8 checks passed
@kenyonj kenyonj deleted the kenyonj-patch-1 branch June 25, 2025 22:48
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR fixes malformed arrays across several topic markdown files by removing trailing commas in array-like lists to ensure proper formatting.

  • Removed trailing commas from the "related" field in topics/ux, topics/stata, and topics/point-and-click.
  • Removed trailing commas from the "aliases" field in topics/mcnp, topics/fuzzing, and topics/crud.

Reviewed Changes

Copilot reviewed 6 out of 24 changed files in this pull request and generated 6 comments.

Show a summary per file
File Description
topics/ux/index.md Removed trailing comma from the "related" array.
topics/stata/index.md Removed trailing comma from the "related" array.
topics/point-and-click/index.md Removed trailing comma from the "related" array.
topics/mcnp/index.md Removed trailing comma from the "aliases" list.
topics/fuzzing/index.md Removed trailing comma from the "aliases" list.
topics/crud/index.md Removed trailing comma from the "aliases" list.

short_description: User experience is about the anticipation of the needs of the user and solutions that fit them.
topic: ux
related: usability, information-architecture, ui, design-patterns, developer-experience, customer-experience, employee-experience, universal-design, accessibility, accessibility-experience,
related: usability, information-architecture, ui, design-patterns, developer-experience, customer-experience, employee-experience, universal-design, accessibility, accessibility-experience
Copy link

Copilot AI Jun 25, 2025

Choose a reason for hiding this comment

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

The removal of the trailing comma in the 'related' list ensures proper array formatting in the YAML file.

Copilot uses AI. Check for mistakes.
created_by: StataCorp
logo: stata.png
related: mata, stata-schemes, data-visualization, data-analysis, data-science,
related: mata, stata-schemes, data-visualization, data-analysis, data-science
Copy link

Copilot AI Jun 25, 2025

Choose a reason for hiding this comment

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

Removing the trailing comma from the 'related' list corrects the array structure and prevents potential parsing errors.

Copilot uses AI. Check for mistakes.
topic: point-and-click
aliases: point-n-click
related: scumm,
related: scumm
Copy link

Copilot AI Jun 25, 2025

Choose a reason for hiding this comment

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

The change removes an unnecessary trailing comma from the 'related' list, ensuring consistent YAML syntax.

Copilot uses AI. Check for mistakes.
@@ -1,5 +1,5 @@
---
aliases: mcnp5, mcnp6,
aliases: mcnp5, mcnp6
Copy link

Copilot AI Jun 25, 2025

Choose a reason for hiding this comment

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

The removal of the trailing comma from the 'aliases' list prevents malformed array issues in the YAML file.

Copilot uses AI. Check for mistakes.
@@ -1,5 +1,5 @@
---
aliases: fuzz-testing, api-fuzzing,
aliases: fuzz-testing, api-fuzzing
Copy link

Copilot AI Jun 25, 2025

Choose a reason for hiding this comment

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

Eliminating the trailing comma from the 'aliases' list resolves potential formatting errors in the YAML declaration.

Copilot uses AI. Check for mistakes.
---
display_name: CRUD
aliases: crudl,
aliases: crudl
Copy link

Copilot AI Jun 25, 2025

Choose a reason for hiding this comment

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

Removing the undesired trailing comma from the 'aliases' list corrects the YAML syntax and ensures proper parsing.

Copilot uses AI. Check for mistakes.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants