Implement JSON validation for all-contributors data

[gandrewgabriel]

Summary

Partially addresses #3549

The script checks for two kinds of problems in the all-contributors JSON: firstly, is the file parsable JSON (to catch problems caused by, for example, missing commas); and secondly, do the contents of the file match the schema required by all-contributors.

The schema I've added only checks the content of the "contributors" property. The other metadata in the JSON file is not validated against a schema, but errors in JSON syntax will be detected anywhere in the file.

I don't know enough about how the CI works, so I haven't made any changes there. When this validation script is ready for action, maybe someone else can add it to the CI pipeline.

What should a reviewer concentrate their feedback on?

• Everything looks ok?
• Scripts are suitable for CI use?

Acknowledging contributors

The following people should be added to the table of contributors in the README file: @gandrewgabriel

[welcome]

💖 Thanks for opening this pull request! 💖 The Turing Way community really appreciates your time and effort to contribute to the project. Please make sure you have read our Contributing Guidelines and filled in our pull request template to the best of your ability.
If you are submitting a new chapter, here are some things that will help get your pull request across the finish line! 🏁

• Check you have removed all lorem ipsums from the chapter template (if you used it)
• Check for any abbreviations or latin phrases (such as "e.g." or "i.e.") in your writing. See our style guide for more information on this topic.
• Make sure you have added your new chapter to the Table of Contents
  
  We have Continuous Integration tests that check the writing style and will help you track down any slip-ups ♻ The Netlify bot will also comment with a preview of the book with your additions so you can see how it will look once it's merged! 🎉
  
  We get a lot of pull requests on this repo, so please be patient and we will get back to you as soon as we can. If we don't acknowledge this pull request after 7 days, feel free to chat to us about it in our Slack workspace.

[netlify]

✅ Deploy Preview for the-turing-way ready!

Name	Link
🔨 Latest commit	8f32576
🔍 Latest deploy log	https://app.netlify.com/sites/the-turing-way/deploys/66630c4c6edbc30008398d59
😎 Deploy Preview	https://deploy-preview-3619--the-turing-way.netlify.app
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

---

To edit notification comments on pull requests, go to your Netlify site configuration.

[aleesteele]

Welcome @gandrewgabriel! Thanks for tagging the @the-turing-way/infrastructure-working-group, @JimMadge to get this reviewed and checked!

[JimMadge]

Thanks @gandrewgabriel ! 🎉

I don't know enough about how the CI works, so I haven't made any changes there. When this validation script is ready for action, maybe someone else can add it to the CI pipeline.

Is that something you would like to learn? It might be possible for someone from the @the-turing-way/infrastructure-working-group to mentor or co-work with you on that.

[sgibson91]

This looks good enough to me. I have a couple of nitpicks about hard-coded paths but that could be addressed in a follow-up PR, so I won't block on it.

I'd be happy to co-work with you to introduce these scripts into CI if you're interested?

[aleesteele]

Hi @sgibson91, @gandrewgabriel, @JimMadge - just checking in with a friendly ping to remind you about this PR! No rush, but our Book Dash week might be a good time to push this forward 🚀

[JimMadge]

Hi @sgibson91, @gandrewgabriel, @JimMadge - just checking in with a friendly ping to remind you about this PR! No rush, but our Book Dash week might be a good time to push this forward 🚀

Absolutely, this can be a target for me during Book Dash!

[JimMadge]

I'm going to look at getting these changes made this week.

[JimMadge]

Just realised that the schema on jsonschema store doesn't validate the contributors objects and the schema added here does. I will try and merge the two here.

Would be good to contribute that extra validation upstream though.

[JimMadge]

🤞 seems to work locally.

[JimMadge]

Merged 😄 SchemaStore/schemastore#3836

[welcome]

Congrats on merging your first pull request! 🎉 We here at The Turing Way are proud of you! 💖 Thank you so much for your contribution 🎁

[JimMadge]

@all-contributors please add @gandrewgabriel for infrastructure, code, test

[allcontributors]

@JimMadge

I've put up a pull request to add @gandrewgabriel! 🎉