-
Notifications
You must be signed in to change notification settings - Fork 12
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
refactor: reuse the logic from snyk-go-parser #37
Conversation
kicking the build for Travis |
523096b
to
9695219
Compare
9695219
to
1e6ab05
Compare
|
||
return Promise.all([ | ||
const result = await Promise.all([ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you please remove the Promise.all
while you are in here?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It's there for a reason: to perform these tasks in parallel. A small saving, but still.
@@ -17,21 +17,20 @@ | |||
"test-functional-windows": "tap -R spec --timeout=300 ./test/*.test.ts -g \"^(?!.*symlink)\"", | |||
"test-system-windows": "tap -R spec --timeout=300 ./test/system/*.test.ts -g \"^(?!.*prometheus)\"", | |||
"test-windows": "npm run test-functional-windows && npm run test-system-windows", | |||
"semantic-release": "semantic-release", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
why is semantic release removed?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The proper style, as taught by @lili2311 , is to use "npx semantic-release" in the .travis.yml, which automatically uses the latest and the greatest version.
🎉 This PR is included in version 1.6.2 🎉 The release is available on: Your semantic-release bot 📦🚀 |
What does this PR do?
Supersedes #36
Since the manifest parsing logic has now been separated into snyk-go-parser, it makes sense for the plugin to re-use that.
Drive-by: robustify tests, clean up the build script.
How should this be manually tested?
Scanning Go projects via CLI.