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

fix: Use actual package manager on test runs #537

Merged
merged 1 commit into from
May 24, 2019

Conversation

lili2311
Copy link
Contributor

@lili2311 lili2311 commented May 24, 2019

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

What does this PR do?

During refactor the packageManager was accidentally hardcoded to 'npm', setting this back

@lili2311 lili2311 self-assigned this May 24, 2019
darscan
darscan previously approved these changes May 24, 2019
@lili2311 lili2311 force-pushed the fix/send-correct-package-manager-analytics branch from be7a4d2 to b86288c Compare May 24, 2019 14:54
@lili2311 lili2311 merged commit 25c03d5 into master May 24, 2019
@snyksec
Copy link

snyksec commented May 24, 2019

🎉 This PR is included in version 1.167.2 🎉

The release is available on:

Your semantic-release bot 📦🚀

@orkamara orkamara deleted the fix/send-correct-package-manager-analytics branch May 26, 2019 15:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants