Skip to content
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

Fix json pre commit errors inside the repo #51

Closed
wants to merge 3 commits into from

Conversation

serdardalgic
Copy link
Contributor

This PR fixes the pre-commit checker errors for json files.

The launch.json file is an autogenerated vscode-go file. VSCode uses Json with Comments(jsonc) mode for its configuration files.

If we can find a way to introduce jsonc check to our pre-commit hooks, we can change back the format of this file.

To read more about JsonC:
https://code.visualstudio.com/docs/languages/json#_json-with-comments

There is also an extension to JSON called Json5, Json for Humans. https://json5.org/ However, according to their website:

it's not intended to be used for machine-to-machine communication.

Please review.

@serdardalgic serdardalgic force-pushed the fix-pre-commit-ci-errors-inside-the-repo branch from 2bd29c5 to 5e51754 Compare November 14, 2022 16:16
drsm79
drsm79 previously approved these changes Nov 14, 2022
@serdardalgic serdardalgic force-pushed the conventional-commit-checker-GH-action branch from 38e5308 to 5745edd Compare November 15, 2022 12:15
Base automatically changed from conventional-commit-checker-GH-action to main November 15, 2022 13:45
pre-commit-ci bot and others added 3 commits November 15, 2022 18:11
This launch.json file is an autogenerated vscode-go file.
VSCode uses Json with Comments(jsonc) mode for its configuration files.

If we can find a way to introduce jsonc check to our pre-commit hooks,
we can change back the format of this file.

To read more about JsonC:
https://code.visualstudio.com/docs/languages/json#_json-with-comments

There is also an extension to JSON called Json5, Json for Humans.
https://json5.org/

Json5 formatter/checkers can be replaced with Json later.
@serdardalgic
Copy link
Contributor Author

I'm closing this PR in favor of #79.
I think the JSON style changes doesn't bring any practical value for now.
The indent change is trivial and not urgent. It can be tackled in another issue.

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.

2 participants