Skip to content
This repository has been archived by the owner on Mar 25, 2021. It is now read-only.

Fix findup when file found in subdirectory #2765

Merged
merged 1 commit into from
May 19, 2017
Merged

Fix findup when file found in subdirectory #2765

merged 1 commit into from
May 19, 2017

Conversation

ajafff
Copy link
Contributor

@ajafff ajafff commented May 14, 2017

PR checklist

  • Addresses an existing issue: #0000
  • New feature, bugfix, or enhancement
    • Includes tests
  • Documentation update

Overview of change:

The new findup implementation missed a path.join(...) and just returned the filename.
This fixes the bug and adds tests to ensure it will not regress.

Is there anything you'd like reviewers to focus on?

CHANGELOG.md entry:

[no-log] because that's not yet released

@andy-hanson
Copy link
Contributor

Is this fixed by #2720?

@ajafff
Copy link
Contributor Author

ajafff commented May 14, 2017

Seems like this fixes it. You can add the tests from this PR to make sure it does

@ajafff ajafff mentioned this pull request May 17, 2017
andy-hanson added a commit to andy-hanson/tslint that referenced this pull request May 19, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants