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

Check if result isn't null #266

Closed
wants to merge 2 commits into from
Closed

Check if result isn't null #266

wants to merge 2 commits into from

Conversation

larsgw
Copy link
Contributor

@larsgw larsgw commented Apr 29, 2018

This may be the case if files are ignored. I do not know what the other cases are, and if they
should be handled differently, but at least they don't cause errors like this.

See #265

This may be the case if files are ignored. I do
not know what the other cases are, and if they
should be handled differently, but at least they
don't cause error like this.

See babel#265
@larsgw
Copy link
Contributor Author

larsgw commented Apr 29, 2018

I'll figure this out later and make a new, proper PR locally instead of relying on CI to test.

@larsgw larsgw closed this Apr 29, 2018
@larsgw
Copy link
Contributor Author

larsgw commented Apr 29, 2018

It seems those failures come from 1b65b35 instead. Checking out master in a fork gives me the same results, and it's the only relevant commit after ac3ed1a (as seen here), and it deals with source maps. However, I already messed up this PR by force-pushing while the PR was closed, so I can't reopen this one anymore.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant