use Spyglass for datapack linting #37
Labels
devops
Issues relating to CI/supporting tooling; e.g. yarn scripts, GitHub workflows
enhancement
New feature or request
we can rely on Spyglass now that v4 is out once CLI linting is implemented
click to view outdated ticket description
we get a little bit of linting in vscode with Datapack Helper Plus, but DHP is pretty buggy and we shouldn't rely on it to actually lint, I don't think.
instead we should probably write a script in-house to do stuff before we run the test server:
we'd want the linting script to ensure to run on specific files (
.mcfunction
,.json
, etc.) and check that:'
instead of"
{Tags:["tag1","tag2"]}
wrong ❌{ Tags: ["tag1", "tag2"] }
correct ✔️we'll probably want more rules than above but that's a decent start.
adding linting ensures our files have consistent styling (like SNBT whitespace)
we'd also want to add this script to thedonedatapack.yml
/resourcepack.yml
workflows so that we run the linting script before we run the test serverThe text was updated successfully, but these errors were encountered: