Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix(list-different): removed the message at the end of execution #288

Merged
merged 3 commits into from Aug 25, 2022

Conversation

dorser
Copy link
Contributor

@dorser dorser commented Jul 20, 2020

What:
This fixes #111
We want to omit the messages at the end of the execution when the --list-different flag is passed.

Why:
The behavior of --list-different is to error out if detects changed files.
The success message that follows this is confusing.

How:
Simple if/else clause

@kylemh
Copy link
Collaborator

kylemh commented Mar 6, 2021

I have no idea why CI didn't run with that merge commit...

@github-actions
Copy link
Contributor

github-actions bot commented May 6, 2021

Stale pull request

@changeset-bot
Copy link

changeset-bot bot commented Aug 25, 2022

🦋 Changeset detected

Latest commit: 97c9c06

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
prettier-eslint-cli Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@codesandbox-ci
Copy link

codesandbox-ci bot commented Aug 25, 2022

This pull request is automatically built and testable in CodeSandbox.

To see build info of the built libraries, click here or the icon next to each commit SHA.

@JounQin JounQin self-assigned this Aug 25, 2022
@JounQin JounQin added the bug label Aug 25, 2022
@JounQin JounQin changed the title fix(list-different): removed the message at the end of execution (#111) fix(list-different): removed the message at the end of execution Aug 25, 2022
@codecov-commenter
Copy link

codecov-commenter commented Aug 25, 2022

Codecov Report

Merging #288 (97c9c06) into master (d8fc957) will not change coverage.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##            master      #288   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files            4         4           
  Lines          165       166    +1     
  Branches        27        29    +2     
=========================================
+ Hits           165       166    +1     
Impacted Files Coverage Δ
src/format-files.js 100.00% <100.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@JounQin JounQin merged commit 4093cb7 into prettier:master Aug 25, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Change success message to warning.
4 participants