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

Don't search for requirements.txt if using pipfile #2309

Merged
merged 2 commits into from Jun 7, 2018

Conversation

Projects
2 participants
@techalchemy
Member

techalchemy commented Jun 6, 2018

  • Also don't search for requirements.txt if taking in a package name
  • Fixes #2222

Signed-off-by: Dan Ryan dan@danryan.co

Don't search for requirements.txt if using pipfile
- Also don't search for requirements.txt if taking in a package name
- Fixes #2222

Signed-off-by: Dan Ryan <dan@danryan.co>

@uranusjr uranusjr merged commit e00fce7 into master Jun 7, 2018

1 of 2 checks passed

continuous-integration/appveyor/pr Waiting for AppVeyor build to complete
Details
buildkite/pipenv Build #283 passed (5 minutes, 41 seconds)
Details

@techalchemy techalchemy deleted the requirementtxt-fix branch Jun 8, 2018

@techalchemy techalchemy moved this from Done to Needs Changelog in 2018.06.x Release Jun 16, 2018

@techalchemy techalchemy moved this from Needs Changelog to Done in 2018.06.x Release Jun 16, 2018

@techalchemy techalchemy moved this from Done to Needs tests in 2018.06.x Release Jun 23, 2018

@techalchemy techalchemy moved this from Needs tests to Done in 2018.06.x Release Jun 24, 2018

techalchemy added a commit that referenced this pull request Jun 24, 2018

Add tests for #2309
Signed-off-by: Dan Ryan <dan@danryan.co>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment