-
Notifications
You must be signed in to change notification settings - Fork 581
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
ESLintBear: Return on empty output #728
Conversation
@sils1297 how do you trigger empty output? |
by providing a good file without errors I think |
wip as its not even passing tests |
damn! Sincerely, Lasse Schuirmann lasse@schuirmann.net On 29 August 2016 at 17:12, Zatreanu Adrian-Gabriel <
|
Hi, it looks like your PR is out of date and needs a rebase. This page may help you to get started on this. (Do not press the update branch button. It's evil.) We also have a quick video tutorial on how to rebase: https://asciinema.org/a/78683. That should help you understand the basics of how it works and what you should be doing. Add the original coala repository as a remote:
Then go to your branch and rebase it onto our master:
Now push it up. You will need to do a force push because you change history here (do not do this on master branches :)):
If you're just looking to fix an issue very quickly and not interested in contributing to coala long term, we can fix it up for you - just tell us :). Ping us on our gitter channel if you need help! |
Oh thanks for the information! I wouldn't be able to do it otherwise, I was really scared to ask :( |
I'm closing this, as it's not needed Can't see any usecase where the patch works |
Cool I'll look out for a reproduction. If I don't manage, this is for the better. |
@sils1297 I just ran ESLint v2.13.1 on a project with no errors and it had no output; seems to be that simple. Edit: Updated to v3.4.0, still no output. Edit 2: Never mind, I'm a sillyhead as well; I wasn't using the arguments coala is using. |
Fixes #727
@underyx can you review this?