Skip to content
This repository has been archived by the owner on Sep 20, 2023. It is now read-only.

Prettier #168

Closed
wants to merge 4 commits into from
Closed

Prettier #168

wants to merge 4 commits into from

Conversation

lastmjs
Copy link
Contributor

@lastmjs lastmjs commented May 10, 2018

Checklist:

  • Test your work and double check you didn't break anything
  • Update any relevant READMEs and docs

Description:

Adding prettier to format code equivalently for all developers. Followed the way origin.js uses prettier, open to make it better or easier to use.

@lastmjs lastmjs requested a review from nick May 10, 2018 22:48
@lastmjs
Copy link
Contributor Author

lastmjs commented May 10, 2018

Looks like there is an issue with the glob patterns, I don't think the ** does what I think it does, it isn't penetrating directories

@gaboesquivel
Copy link

see #189

@lastmjs
Copy link
Contributor Author

lastmjs commented May 17, 2018

Should this be closed in lieu of #189 ?

@micahalcorn
Copy link
Member

Yes, I'm going to close this in favor of #189 for now. Single quotes ftw. 💪

@micahalcorn micahalcorn deleted the prettier branch August 10, 2018 20:50
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants