Skip to content

Add JSON linting#184

Open
j616 wants to merge 2 commits intomainfrom
jamessa-jsonLint
Open

Add JSON linting#184
j616 wants to merge 2 commits intomainfrom
jamessa-jsonLint

Conversation

@j616
Copy link
Copy Markdown
Contributor

@j616 j616 commented Apr 16, 2026

Details

  • Add a JSON linter to the API directory makefile
  • Add a make target for conforming JSON files to the linter
  • Conform JSON files

Run that shows linter finding errors: https://github.com/bbc/tams/actions/runs/24517163451/job/71664365905?pr=184

Jira Issue (if relevant)

Jira URL: https://jira.dev.bbc.co.uk/browse/CLOUDFIT-5543

Related PRs

Where appropriate. Indicate order to be merged.

Submitter PR Checks

(tick as appropriate)

  • PR completes task/fixes bug
  • API version has been incremented if necessary
  • ADR status has been updated, and ADR implementation has been recorded
  • Documentation updated (README, etc.)
  • PR added to Jira Issue (if relevant)
  • Follow-up stories added to Jira

Reviewer PR Checks

(tick as appropriate)

  • PR completes task/fixes bug
  • Design makes sense, and fits with our current code base
  • Code is easy to follow
  • PR size is sensible
  • Commit history is sensible and tidy

Info on PRs

The checks above are guidelines. They don't all have to be ticked, but they should all have been considered.

@j616 j616 requested a review from a team as a code owner April 16, 2026 14:49
@j616 j616 force-pushed the jamessa-jsonLint branch from cf68ebc to b69a692 Compare April 16, 2026 14:52
@j616 j616 force-pushed the jamessa-jsonLint branch from 8405c9f to 7405566 Compare April 17, 2026 13:44
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