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

Add prettier as eslint plugin #24

Merged
merged 7 commits into from
Sep 19, 2017
Merged

Conversation

wacii
Copy link
Collaborator

@wacii wacii commented Sep 11, 2017

Compares prettier output to actual code when yarn lint is run, reporting the differences as errors. yarn prettier applies the changes using the --fix flag.

Rodrigo Solis and others added 4 commits September 11, 2017 00:51
* develop:
  fix: update babel + add missing plugin
  fix: simplify check for better compat
  fix: add eslint disable to discard and effect
  fix: removes unsafe check
  fix: eslint errors
  fix: eslint error with --fix flag
  fix: ignore tests
  feat: changes eslint config + updates dependencies Creates new eslint config based on Airbnb`s one. Updates eslint versions and removes unused dependecies
  Payload of action did not fit to reducer anymore
  fix: restores retrySchedule and retryCount on rehydrate + uses initialState instead of magic values
  fix: squashes persisted netInfo with state`s netInfo
* develop:
  fix: package.json errors
Involved enough to warrant tests. Separating them into modules allows
the integration point to be tested more easily as well.
@wacii
Copy link
Collaborator Author

wacii commented Sep 11, 2017

See #22.

@sorodrigo
Copy link
Owner

Awesome!! Thanks for all the help. Do you have a twitter account or email I can reach you at? Seems like something broke when I merged the other PR, and now travis is complaining. Could you take a look a it so we can merge this? Thanks again, amazing job!

@sorodrigo
Copy link
Owner

Another thing, I accidentally merged last PR into master. I'm trying to enforce git flow, I'm not reverting last PR, its too much hassle, but for the next PRs could you point them at the develop branch please. 🙏

@sorodrigo sorodrigo changed the base branch from master to develop September 19, 2017 03:22
@wacii
Copy link
Collaborator Author

wacii commented Sep 19, 2017

Oh right, I didn't really notice the develop branch, sorry about that. I'll direct my PRs there from now on.

You can reach me at [email protected]. I do not twitter.

I'll take a look at the error now.

@wacii
Copy link
Collaborator Author

wacii commented Sep 19, 2017

Just needed to apply prettier to PR#23 as any differences register as lint errors now.

@sorodrigo sorodrigo merged commit 1c6e94c into sorodrigo:develop Sep 19, 2017
@wacii wacii deleted the add-prettier branch February 20, 2018 19:34
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.

2 participants