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

Prettier-ize & lint Bowser and generated projects #214

Merged
merged 11 commits into from
Jul 4, 2019
Merged

Conversation

bryanstearns
Copy link
Member

I made a new project and found that out-of-the-box it had lint complaints and formatting that prettier was unhappy with. So:

  • I set up prettier on bowser itself, and bumped a few dependency versions
  • Fixed lint complaints and prettier-ized Bowser,
  • Updated the boilerplate's package.json dependencies and configuration for prettier and eslint
  • Added a lint and format step to boilerplate.js
  • Added to the Bowser generated-app test to make sure new projects format & lint cleanly. This mades the tests run a little longer, triggering a new complaint from Jest - to fix that, we now run the tests in-band.
  • Fixed the remaining boilerplate complaints to make the tests pass.

Use same settings as generated projects
Also lint js files at root
Don't bother prettying ignite.json:

Ignite (indirectly thru gluegun) uses fs-jetback to write this file;
there's no easy way to tell it to make sure to write a newline on the
end, and this file is written after Bowser finishes so we can't fix it.

Rather than leave this as a difference that'll be created the first
time someone runs `yarn format` in the new project, we'll ignore it.
@bryanstearns bryanstearns changed the title Prettier-ize & lint bowser & generated projects Prettier-ize & lint Bowser and generated projects Jun 28, 2019
Copy link
Member

@ryanlntn ryanlntn left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

😎

@bryanstearns bryanstearns merged commit 381a900 into master Jul 4, 2019
infinitered-circleci pushed a commit that referenced this pull request Jul 4, 2019
# [4.5.0](v4.4.5...v4.5.0) (2019-07-04)

### Features

* Prettier-ize & lint Bowser and generated projects ([#214](#214)) ([381a900](381a900))
@infinitered-circleci
Copy link

🎉 This PR is included in version 4.5.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants