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.

Sign up for GitHub

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

Jump to bottom

[FE] Simplify Eslint config #123

Merged
merged 1 commit into from
Nov 26, 2020
Merged

[FE] Simplify Eslint config #123

merged 1 commit into from
Nov 26, 2020

Conversation

whotake
Copy link
Contributor

@whotake whotake commented Nov 24, 2020 •
edited
Loading

This PR aims to simplify the current Eslint config.

  • Use adapted to Typescript version of eslint-config-airbnb. No longer needed to fix them in our config file
  • Cleanup the current Eslint config
  • Fix new Eslint errors. Mostly no-param-reassign errors
  • Add to eslintignore files generated by OpenAPI

@whotake whotake merged commit 1a7aed1 into provectus:master Nov 26, 2020
@whotake whotake deleted the enhancement/simplift-eslint-config branch November 26, 2020 10:52
javalover123 pushed a commit to javalover123/kafka-ui that referenced this pull request Dec 7, 2022
@whotake
Merge pull request provectus#123 from whotake/enhancement/simplift-es…
722eb52 
…lint-config

[FE] Simplify Eslint config
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@workshur workshur workshur approved these changes

@soffest soffest soffest approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@whotake @workshur @soffest