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

fix: Use fork of prettier-eslint with cwd fix #564

Merged

Conversation

lewisl9029
Copy link
Contributor

@lewisl9029 lewisl9029 commented Nov 4, 2019

Using a temporary fork for prettier-eslint due to maintainer having been unresponsive. Feels
warranted as this is a showstopping bug affecting many users.

Fork uses the changes in this PR: prettier/prettier-eslint#242

Implemented using yarn's alias feature: https://yarnpkg.com/lang/en/docs/cli/add/#toc-yarn-add-alias

fix #505

Using a temporary fork for prettier-eslint due to maintainer having been unresponsive. Feels
warranted as this is a showstopping bug affecting many users.

fix prettier#505
@lewisl9029 lewisl9029 force-pushed the use-fork-of-prettier-eslint-with-cwd-fix branch from 5f81493 to 2e0b684 Compare November 4, 2019 10:11
@codecov-io
Copy link

Codecov Report

Merging #564 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@          Coverage Diff          @@
##           master   #564   +/-   ##
=====================================
  Coverage       0%     0%           
=====================================
  Files           1      1           
  Lines          61     61           
  Branches        7      7           
=====================================
  Misses         54     54           
  Partials        7      7

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 394be4d...2e0b684. Read the comment docs.

Copy link
Collaborator

@robwise robwise left a comment

Choose a reason for hiding this comment

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

Yep, agreed, thanks!

@robwise robwise self-assigned this Nov 4, 2019
@robwise robwise merged commit 63326f7 into prettier:master Nov 4, 2019
lewisl9029 added a commit to lewisl9029/prettier-atom that referenced this pull request Nov 4, 2019
…lint'

This is an alternative implemnetation of prettier#564, which
used yarn alias to avoid making code changes. Unfortunately apm is tightly coupled to npm, which
doesn't support the alias feature.

fix prettier#565
lewisl9029 added a commit to lewisl9029/prettier-atom that referenced this pull request Nov 4, 2019
This is an alternative implementation of prettier#564, which
used yarn alias to avoid making code changes. Unfortunately apm is tightly coupled to npm, which
doesn't support the alias feature.

fix prettier#565
lewisl9029 added a commit to lewisl9029/prettier-atom that referenced this pull request Nov 5, 2019
This is an alternative implementation of prettier#564, which
used yarn alias to avoid making code changes. Unfortunately apm is tightly coupled to npm, which
doesn't support the alias feature.

fix prettier#565
lewisl9029 added a commit to lewisl9029/prettier-atom that referenced this pull request Nov 5, 2019
This is an alternative implementation of prettier#564, which
used yarn alias to avoid making code changes. Unfortunately apm is tightly coupled to npm, which
doesn't support the alias feature.

fix prettier#565
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.

Failed to load plugin 'prettier' declared in 'CLIOptions': Cannot find module 'eslint-plugin-prettier'
3 participants