Skip to content
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

chore: set up linting for typescript definitions #890

Merged
merged 2 commits into from May 25, 2020

Conversation

rchl
Copy link
Contributor

@rchl rchl commented May 24, 2020

It's hard to contribute to the project when there are linting and parsing errors shown by the editor everywhere so I hope you'll consider accepting this.

@rchl rchl closed this May 24, 2020
@rchl rchl reopened this May 24, 2020
@rchl
Copy link
Contributor Author

rchl commented May 25, 2020

Is the CI failure related to my changes?
It says: This version of ChromeDriver only supports Chrome version 81
It doesn't look like my change changes any dependencies related to that.

@kazupon
Copy link
Owner

kazupon commented May 25, 2020

Thank you for your PR!
That's great work! 👍

It says: This version of ChromeDriver only supports Chrome version 81

I'll try to fix chrome driver.

@kazupon kazupon added the Type: Improvement Includes backwards-compatible fixes label May 25, 2020
@kazupon kazupon merged commit aef0bfa into kazupon:v8.x May 25, 2020
@rchl rchl deleted the chore/ts-linting branch May 25, 2020 14:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
PR: merged Type: Improvement Includes backwards-compatible fixes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants