Skip to content

Add JSON validation to GitHub Action#1481

Merged
DHardy-WMF merged 1 commit intoWikipediaLibrary:masterfrom
DHardy-WMF:dhardy/T404882
Sep 29, 2025
Merged

Add JSON validation to GitHub Action#1481
DHardy-WMF merged 1 commit intoWikipediaLibrary:masterfrom
DHardy-WMF:dhardy/T404882

Conversation

@DHardy-WMF
Copy link
Contributor

@DHardy-WMF DHardy-WMF commented Sep 29, 2025

Description

This is a follow-up patch for T404882. A new step was added to prevent invalid JSON syntax in pull requests, which is determined by a non-zero exit code returned from the jq command.

Rationale

Prevents junk syntax from being added to automated pull requests.

Phabricator Ticket

https://phabricator.wikimedia.org/T404882

How Has This Been Tested?

Forked branch

Screenshots of your changes (if appropriate):

Types of changes

What types of changes does your code introduce? Add an x in all the boxes that apply:

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Minor change (fix a typo, add a translation tag, add section to README, etc.)

This is a follow-up patch for T404882. A new step was added to prevent
invalid JSON syntax in pull requests, which is determined by a non-zero
exit code returned from the jq command.
@DHardy-WMF DHardy-WMF merged commit 6337471 into WikipediaLibrary:master Sep 29, 2025
4 checks passed
@DHardy-WMF DHardy-WMF mentioned this pull request Sep 29, 2025
4 tasks
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.

1 participant