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

Jump to bottom

Send warnings to STDERR, not STDOUT #2082

Closed
docwhat opened this issue Dec 11, 2018 · 4 comments
Closed

Send warnings to STDERR, not STDOUT #2082

docwhat opened this issue Dec 11, 2018 · 4 comments
Assignees
@ljharb
Labels
bug

Comments

@docwhat
Copy link

docwhat commented Dec 11, 2018

In issue #1955 the warning is being sent to STDOUT which is messing up editors by adding the warning to the top of the file.

All warnings and errors should be sent to STDERR.
@ljharb
Copy link
Member

ljharb commented Dec 11, 2018

Good call, this should be fixed.

@ljharb ljharb added the bug label Dec 11, 2018
@ljharb ljharb self-assigned this Dec 11, 2018
@ljharb ljharb closed this as completed in 8ef86c5 Dec 11, 2018
docwhat added a commit to docwhat/website that referenced this issue Dec 11, 2018
@docwhat
workaround for bugs in prettier-eslint
f8fa163 
- jsx-eslint/eslint-plugin-react#2082 is just the symptom that is causing
my editor to add "Warnings" to the top of my .js files.

- prettier/prettier-eslint#194 is the actual bug; prettier-eslint isn't
passing the `baseConfig` to eslint.
docwhat added a commit to docwhat/website that referenced this issue Dec 12, 2018
@docwhat
workaround for bugs in prettier-eslint
974274d 
- jsx-eslint/eslint-plugin-react#2082 is just the symptom that is causing
my editor to add "Warnings" to the top of my .js files.

- prettier/prettier-eslint#194 is the actual bug; prettier-eslint isn't
passing the `baseConfig` to eslint.
@ahakanbaba
Copy link

Could there be a new release including this fix ?

@renovate renovate bot mentioned this issue Dec 28, 2018
Merged
1 task
@renovate-bot renovate-bot mentioned this issue Dec 28, 2018
Closed
1 task
@renovate renovate bot mentioned this issue Dec 28, 2018
Merged
1 task
@ljharb
Copy link
Member

ljharb commented Dec 28, 2018

v7.12.0 is released.

This was referenced Dec 28, 2018
Merged
Merged
Merged
Merged
Merged
Closed
Open
Closed
Merged
Closed
Open
Open
Merged
Open
Open
Merged
Closed
Merged
This was referenced Jan 4, 2019
Merged
Closed
Open
Merged
Closed
Merged
Closed
Merged
Merged
Closed
Closed
Merged
Closed
Merged
Closed
Merged
Merged
Merged
@ghost ghost mentioned this issue Jan 12, 2019
Merged
1 task
This was referenced Jan 13, 2019
Merged
Merged
Merged
Merged
Merged
Merged
This was referenced Aug 17, 2019
Open
Open
This was referenced Sep 22, 2019
Open
Open
@snyk-bot snyk-bot mentioned this issue Sep 30, 2019
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@ljharb ljharb

Labels
bug
Milestone
No milestone
Development

No branches or pull requests
4 participants
@docwhat @ljharb @alexzherdev @ahakanbaba