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

feat(options): add prettierLast option for prettier-eslint #110

Merged
merged 1 commit into from
Apr 2, 2017
Merged

feat(options): add prettierLast option for prettier-eslint #110

merged 1 commit into from
Apr 2, 2017

Conversation

spudly
Copy link
Contributor

@spudly spudly commented Mar 31, 2017

Adds a new Run Prettier Last option (see prettier/prettier-eslint#58 and prettier/prettier-eslint#59).

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.

@spudly This is great code! One minor thing and then we can merge!

helpers.shouldUseEslint.mockImplementation(() => true);
// $FlowFixMe
helpers.getCurrentFilePath.mockImplementation(() => 'foo.js');
helpers.getPrettierEslintOptions.mockImplementation(() => ({ prettierLast: true }));
Copy link
Collaborator

Choose a reason for hiding this comment

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

we just need to add a // $FlowFixMe here to pass flow

Copy link
Contributor Author

Choose a reason for hiding this comment

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

done.

@spudly
Copy link
Contributor Author

spudly commented Mar 31, 2017

we just need to add a // $FlowFixMe here to pass flow

done.

@codecov-io
Copy link

codecov-io commented Mar 31, 2017

Codecov Report

Merging #110 into master will increase coverage by 0.4%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff            @@
##           master     #110     +/-   ##
=========================================
+ Coverage   78.57%   78.98%   +0.4%     
=========================================
  Files           6        6             
  Lines         154      157      +3     
=========================================
+ Hits          121      124      +3     
  Misses         33       33
Impacted Files Coverage Δ
src/helpers.js 100% <100%> (ø) ⬆️
src/executePrettier.js 100% <100%> (ø) ⬆️

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 5b38dff...0f2714d. Read the comment docs.

@robwise
Copy link
Collaborator

robwise commented Mar 31, 2017

@spudly awesome, can you squash down that commit?

@spudly
Copy link
Contributor Author

spudly commented Apr 1, 2017

squashed. also added myself as contributor the right way, as I had done it manully before.

@robwise robwise merged commit 8e515b4 into prettier:master Apr 2, 2017
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.

None yet

3 participants