JSON lint - Githubissues

gophertown / gophertown-data

A listing of gophers

19 stars 37 forks source link

JSON lint #27

Open nathany opened 9 years ago

nathany commented 9 years ago

Maybe we should add a JSON linter on pull requests via CI? See #23.

dgryski commented 9 years ago

The current rustaceans code contains a webhook that verifies that the pull request modifies a single file that matches your github username, and if so merges it automatically. If we added linting, we could so something similar.

dgryski commented 9 years ago

We could also check for other formatting errors:

twitter handle doesn't start with @
blog link starts with http://, https://', or //

We could also fix some of these client-side at template render time.

nathany commented 9 years ago

Or fix up on the server side when preparing the JSON.