Skip to content

Fix: format JSON using JSON.NET & test #1292

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.

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

Merged
merged 4 commits into from
Dec 26, 2024
Merged

Fix: format JSON using JSON.NET & test #1292

merged 4 commits into from
Dec 26, 2024

Conversation

david-allison
Copy link
Owner

Previously, we had a number of manifests which could not be opened in Python:

  • newlines in the strings
  • trailing commas

So we now confirm that our manifests conform strictly to the JSON spec

Fixes #1291

using provided unit test

Python did not open a number of files

Issue 1291
Using provided unit test

Python did not open a number of files

Issue 1291
so they can be opened using Pyton

Issue 1291
@david-allison david-allison merged commit a40eb58 into master Dec 26, 2024
1 check passed
@david-allison david-allison deleted the json branch December 26, 2024 17:49
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.

Normalize/validate manifests
1 participant