Skip to content

fix: errors not being displayed in the correct files#700

Merged
mgechev merged 1 commit intomgechev:masterfrom
rafaelss95:fix-regression-deprecation
Aug 13, 2018
Merged

fix: errors not being displayed in the correct files#700
mgechev merged 1 commit intomgechev:masterfrom
rafaelss95:fix-regression-deprecation

Conversation

@rafaelss95
Copy link
Collaborator

Fixes #692 #699.

It was a regression caused by PR #587. I'm not sure how we can add tests to prevent errors like this from popping up.

@rafaelss95 rafaelss95 requested review from mgechev and wKoza August 13, 2018 03:25
Copy link
Owner

@mgechev mgechev left a comment

Choose a reason for hiding this comment

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

Good catch 👍

@mgechev mgechev merged commit db3cf5a into mgechev:master Aug 13, 2018
@rafaelss95 rafaelss95 deleted the fix-regression-deprecation branch August 13, 2018 04:26
@jaufgang
Copy link

Thank you @rafaelss95 for fixing this.

@mgechev, could you please publish a new release ASAP with this fix merged in? The bug is actually mangling code, so getting a new release out quickly will help prevent other developers from the ensuing headaches of encountering this bug. Thanks!

@mayfieldiv
Copy link

Glad to see this will be in the next release. I updated my dependencies today to find the linter mangling TS files as others mentioned.

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.

4 participants