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

Fix eslint prettier #518

Merged
merged 4 commits into from
Jun 18, 2019

Conversation

asprouse
Copy link
Contributor

@asprouse asprouse commented Jun 15, 2019

What did you implement:

Closes #515

How did you implement it:

  • Configure prettier-eslint to format code, remove eslint-config-prettier
  • Update prettier config to be closer to existing code
  • Move npm test command out of lint-staged to .huskyrc. This was causing errors trying to run staged files as if they were tests. All tests should be run not just the staged test files.
  • Reformat code with prettier-eslint

How can we verify it:

  • @HyperBrain Look at the last commit in this PR and make sure that the code style is acceptable since introducing prettier normalizes spacing, semicolons, line-length, etc. I tried to make the config as close to the current style preferences but the current ESLint rules weren't tight enough to enforce consistency code being reformatted was inevitable.
  • run npm run format then npm run eslint and verify there are no formatting lint issues
  • Try committing some code then run npm run eslint and ensure there are no issues.

Todos:

  • Write tests
  • Write documentation
  • Fix linting errors
  • Make sure code coverage hasn't dropped (Some existing lines were split that contained uncovered code this increased the number of uncovered lines but didn't affect branches or functions).
  • Provide verification config / commands / resources
  • Enable "Allow edits from maintainers" for this PR
  • Update the messages below

Is this ready for review?: YES
Is it a breaking change?: NO

@HyperBrain
Copy link
Member

Looks good to me. Now it should be much easier again to contribute without anyone being annoyed by uncontrollable changes.

@asprouse
Copy link
Contributor Author

Great! Merge away!

@HyperBrain
Copy link
Member

HyperBrain commented Jun 17, 2019 via email

@HyperBrain HyperBrain merged commit 2139384 into serverless-heaven:master Jun 18, 2019
@HyperBrain
Copy link
Member

Merged. Feel free to get the other PR to fit now ;-)

jamesmbourne pushed a commit to jamesmbourne/serverless-webpack that referenced this pull request Oct 15, 2019
@miguel-a-calles-mba miguel-a-calles-mba added this to the 5.3.2 milestone Apr 27, 2020
@miguel-a-calles-mba miguel-a-calles-mba mentioned this pull request Apr 29, 2020
7 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

prettier and eslint configs conflict and break tests
3 participants