Skip to content

Commit

Permalink
πŸ’… Attempt to install prettier packages, too (#4)
Browse files Browse the repository at this point in the history
* πŸ’… Attempt to intall prettier packages, too

We're seeing some weird prettier errors when linting a project that has eslint configured to run prettier. My theory is that different prettier versions are being installed depending on peer dependencies vs top-level dependencies. 🀞

* Try to nstall babel dependencies, too

The same strategy that was used for eslint, applied to babel. Parsing has
an impact on linting, so I'm curious to see if this makes the remaining
3 errors on my example branch go away.

* Revert "Try to nstall babel dependencies, too"

This reverts commit 2df54b0.
  • Loading branch information
danott authored Dec 3, 2019
1 parent 19c9e58 commit 621bb5a
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion main.js
Original file line number Diff line number Diff line change
Expand Up @@ -49,7 +49,7 @@ async function installEslintPackagesAsync () {
const yarn = await getYarn()

const versions = yarn.data.trees
.filter(p => p.name.match(/eslint/))
.filter(p => p.name.match(/eslint/) || p.name.match(/prettier/))
.map(p => p.name)

await io.mv('package.json', 'package.json-bak')
Expand Down

0 comments on commit 621bb5a

Please sign in to comment.