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
feat(nx-plugin): add plugin-lint executor #9697
Conversation
This pull request is being automatically deployed with Vercel (learn more). 🔍 Inspect: https://vercel.com/nrwl/nx-dev/GcNyL8LkiNa8EEue9ENFAiBRT5Ef [Deployment for 553f8e3 canceled] |
dabb60f
to
173a021
Compare
173a021
to
e224b57
Compare
e224b57
to
1eb8700
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Awesome job 👏🏻
packages/nx-plugin/src/executors/plugin-lint/plugin-lint.impl.ts
Outdated
Show resolved
Hide resolved
packages/nx-plugin/src/executors/plugin-lint/plugin-lint.impl.ts
Outdated
Show resolved
Hide resolved
20ff401
to
0ad1d23
Compare
0ad1d23
to
0c6ede6
Compare
0c6ede6
to
2e684c7
Compare
2e684c7
to
553f8e3
Compare
553f8e3
to
030bb71
Compare
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
030bb71
to
e206093
Compare
60346ae
to
d8d771e
Compare
d8d771e
to
5b34c95
Compare
5b34c95
to
34ce230
Compare
c3e5993
to
3f6a774
Compare
3f6a774
to
f8b4231
Compare
packages/nx-plugin/src/generators/plugin-lint-checks/generator.ts
Outdated
Show resolved
Hide resolved
packages/nx-plugin/src/generators/plugin-lint-checks/generator.ts
Outdated
Show resolved
Hide resolved
function setupVsCodeLintingForJsonFiles(host: Tree) { | ||
let existing: Record<string, unknown> = {}; | ||
if (host.exists('.vscode/settings.json')) { | ||
existing = readJson<Record<string, unknown>>(host, '.vscode/settings.json'); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Use updateJson
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't want to use updateJson
here in case the settings.json file doesn't exist. I updated the method to actually write the file back though, and log out some info explaining the file if it is created.
packages/nx-plugin/src/generators/plugin-lint-checks/generator.ts
Outdated
Show resolved
Hide resolved
packages/nx-plugin/src/generators/plugin-lint-checks/schema.json
Outdated
Show resolved
Hide resolved
packages/nx-plugin/src/generators/plugin-lint-checks/generator.ts
Outdated
Show resolved
Hide resolved
7123f23
to
be7344c
Compare
be7344c
to
2c8c887
Compare
2c8c887
to
1b06769
Compare
1b06769
to
029c530
Compare
Co-authored-by: Giora Guttsait <giora111@gmail.com>
Co-authored-by: Giora Guttsait <giora111@gmail.com>
029c530
to
21bf25a
Compare
This pull request has already been merged/closed. If you experience issues related to these changes, please open a new issue referencing this pull request. |
Current Behavior
Nothing is provided to validate common json files for Nx plugins, leaving authors to rely on e2e tests to catch errors.
Expected Behavior
Generated nx plugins setup eslint checks to validate common js files. Checks include
/src/my-plugin/generators/app/app#compat
)/src/my-plugin/generators/app/app#compat
)Related Issue(s)
Fixes #