Skip to content

Conversation

@edieblu
Copy link
Contributor

@edieblu edieblu commented Jan 27, 2019

Something like this? I've added editorconfig, eslint and prettier (I haven't actually run it against all the files yes). Let me know what you think. And if you're busy let me know too and I can ask someone else for review :)

@edieblu edieblu requested a review from jalada January 27, 2019 14:17
@jalada
Copy link
Contributor

jalada commented Jan 29, 2019

Happy with editorconfig. I'll need a bit more time to check out eslint, prettier & husky. If anyone else has used those before and so can confidently say yay or nay, happy to defer to them.

@edieblu
Copy link
Contributor Author

edieblu commented Jan 29, 2019

Happy with editorconfig. I'll need a bit more time to check out eslint, prettier & husky. If anyone else has used those before and so can confidently say yay or nay, happy to defer to them.

Thanks @jalada! @sztupi knows maybe?

@jalada
Copy link
Contributor

jalada commented Feb 15, 2019

I've ran all the JS through prettier! I've had a look and prettier looks great, and Husky makes sense.

eslint isn't configured to run as part of husky though. Also, eslint and prettier are fighting; eslint, in its current configuration, tries to change all double quotes to single quotes but prettier changes them to double quotes.

You can see this by doing:

$ npx eslint --fix "**/*.js"

and then checking the git diff.

Maybe we need to configure eslint some more?

@edieblu
Copy link
Contributor Author

edieblu commented Feb 16, 2019

I've ran all the JS through prettier! I've had a look and prettier looks great, and Husky makes sense.

eslint isn't configured to run as part of husky though. Also, eslint and prettier are fighting; eslint, in its current configuration, tries to change all double quotes to single quotes but prettier changes them to double quotes.

You can see this by doing:

$ npx eslint --fix "**/*.js"

and then checking the git diff.

Maybe we need to configure eslint some more?

Hello :) I've updated the settings, the two should now play together nicely. There are some eslint issues that need fixing (I've added the airbnb linter which is quite strict), before I go fixing those, let me know what you think of this setup.

@edieblu edieblu requested review from sztupi and removed request for jalada and singloon February 20, 2019 08:05
@edieblu
Copy link
Contributor Author

edieblu commented Feb 20, 2019

Hey @sztupi these are the changes from yesterday's meetup, and lots of formatting with prettier :)

@edieblu edieblu mentioned this pull request Feb 24, 2019
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.

3 participants