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

feat: support setup.py as a return target file #91

Merged
merged 1 commit into from
Dec 1, 2019

Conversation

dkontorovskyy
Copy link
Contributor

@dkontorovskyy dkontorovskyy commented Nov 28, 2019

  • Ready for review
  • Follows CONTRIBUTING rules
  • Reviewed by Snyk internal team

What does this PR do?

Add return targetFile from plugin setup.py if it's the project manifest

@dkontorovskyy dkontorovskyy requested a review from a team as a code owner November 28, 2019 15:37
@dkontorovskyy dkontorovskyy force-pushed the feat/add-setup-py-target-file branch from 1d1d0fb to 59ee45b Compare November 28, 2019 15:37
@dkontorovskyy dkontorovskyy self-assigned this Nov 28, 2019
@ghost ghost requested review from miiila and orsagie November 28, 2019 15:37
@claassistantio
Copy link

claassistantio commented Nov 28, 2019

CLA assistant check
All committers have signed the CLA.

@dkontorovskyy dkontorovskyy requested review from robcresswell and removed request for miiila November 28, 2019 15:38
@dkontorovskyy dkontorovskyy force-pushed the feat/add-setup-py-target-file branch from 59ee45b to 2870c3f Compare December 1, 2019 10:50
@@ -18,8 +18,9 @@ export function getMetaData(
name: 'snyk-python-plugin',
runtime: output.replace('\n', ''),
// specify targetFile only in case of Pipfile
targetFile:
path.basename(targetFile) === 'Pipfile' ? targetFile : undefined,
targetFile: path.basename(targetFile).match(/^Pipfile|setup\.py$/)
Copy link
Contributor

Choose a reason for hiding this comment

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

this also matches Pipfilexyz and blahsetup.py

@dkontorovskyy dkontorovskyy force-pushed the feat/add-setup-py-target-file branch from 2870c3f to 90a4140 Compare December 1, 2019 11:45
@dkontorovskyy dkontorovskyy merged commit c145a7f into master Dec 1, 2019
@dkontorovskyy dkontorovskyy deleted the feat/add-setup-py-target-file branch December 1, 2019 12:57
@snyksec
Copy link

snyksec commented Dec 1, 2019

🎉 This PR is included in version 1.14.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

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

Successfully merging this pull request may close these issues.

4 participants