-
-
Notifications
You must be signed in to change notification settings - Fork 11
chore(dev): bump @wdio/eslint and fix lint issues from new rules #507
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
base: main
Are you sure you want to change the base?
Conversation
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.
Same here, the formatting of the types seems off, was this autogenerated?
Yes autogenerated with I see the issue that you are pointing to though, so I did some digging and this is what I figured out as the cause and the potential solution: @wdio/eslint 0.0.5 used ESLint's core A good long-term fix would be to add I am thinking about a change like the following for @wdio/eslint:
We'll still need to adjust some formatting in node-geckodriver and node-edgedriver, but without lint rules for JSDoc blocks we are going to have to manually check those comment blocks for misalignments. |
I think another option might be to just drop the style lint rules, and add prettier and have it format the blocks and such. That can potentially resolve webdriverio/eslint#30 as well. I can work on PRs for this option as well if you like. |
I manually added indentations to make them look right. Future manual work is probably not going to be needed once webdriverio/eslint#30 is resolved. |
Dependabot was unable to auto-merge @wdio/eslint updates due to lint rule changes. This commit manually bumps the version and resolves indentation and formatting issues required by the new config.
Rebased to resolve conflicts |
Dependabot was unable to auto-merge @wdio/eslint updates due to lint rule changes. This commit manually bumps the version and resolves indentation and formatting issues required by the new config.