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
Enforce nextcloud code style | Adding eslint linting #554
Conversation
ba61784
to
bb02e2e
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.
See comment
"overrides": [ | ||
{ | ||
"files": ["tests/*.js"], | ||
"rules": { | ||
"n/no-unpublished-import": "off", | ||
// Todo: convert tests to TS then drop overrides below | ||
"n/no-missing-import": "off", | ||
"import/no-unresolved": "off", | ||
"import/extensions": "off" | ||
} | ||
}, | ||
{ | ||
"files": ["*.config.*js"], | ||
"rules": { | ||
"n/no-unpublished-import": "off" | ||
} | ||
} | ||
] |
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.
Would be amazing to upstream that to the esling config 🚀
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.
"cypress/**/*.js", "cypress/**/*.ts", "*.cy.js", "*.cy.ts"
too (cypress)
"*.config.*js", "*.config.*ts"
bb02e2e
to
a93853d
Compare
Signed-off-by: Ferdinand Thiessen <rpm@fthiessen.de>
Signed-off-by: Ferdinand Thiessen <rpm@fthiessen.de>
Signed-off-by: Ferdinand Thiessen <rpm@fthiessen.de>
…lders Signed-off-by: Ferdinand Thiessen <rpm@fthiessen.de>
Signed-off-by: Ferdinand Thiessen <rpm@fthiessen.de>
Signed-off-by: Ferdinand Thiessen <rpm@fthiessen.de>
a93853d
to
e850495
Compare
Now that our eslint config supports TypeScript we would make use of it, what do you think?
I pushed the fixes for problems found as different commits to make reviewing the changes easier (you could review a single commit a time and compare with the description from the commit message), as the changes are quite huge.