Skip to content

Commit

Permalink
chore: simplify plugin lint
Browse files Browse the repository at this point in the history
  • Loading branch information
Zamiell committed Sep 20, 2024
1 parent 28a1473 commit cacd456
Show file tree
Hide file tree
Showing 3 changed files with 6 additions and 3 deletions.
5 changes: 4 additions & 1 deletion knip.config.js
Original file line number Diff line number Diff line change
Expand Up @@ -5,6 +5,10 @@

/** @type {import("knip").KnipConfig} */
const config = {
// Ignore all dependencies in the root. (This is checked by the `lintMonorepoPackageJSONs` helper
// function.)
ignoreDependencies: [".+"],

workspaces: {
"packages/*": {},
"packages/docs": {
Expand Down Expand Up @@ -57,7 +61,6 @@ const config = {
},
"packages/isaacscript-spell": {},
},
ignoreDependencies: ["eslint-plugin-isaacscript"],
};

export default config;
2 changes: 1 addition & 1 deletion packages/eslint-plugin-isaacscript/scripts/lint.ts
Original file line number Diff line number Diff line change
Expand Up @@ -50,11 +50,11 @@ async function checkGenerateChangedFiles() {
if (oldFileContents !== newFileContents) {
changed = true;
echo(`The "generate.ts" script changed the following file: ${filePath}`);
echo('Run: "npm run generate"');
}
}

if (changed) {
echo('Run "npm run generate" and commit the changes.');
exit(1);
}
}
2 changes: 1 addition & 1 deletion scripts/lint.ts
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,7 @@ await lintScript(async () => {
$`prettier --log-level=warn --check .`,

// Use Knip to check for unused files, exports, and dependencies.
// TODO
// TODO: https://discord.com/channels/1143209786612125696/1285734848224432138
/// $`knip --no-progress`,

// Use CSpell to spell check every file.
Expand Down

0 comments on commit cacd456

Please sign in to comment.