update pkg conf #1
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
### Rationale Fixes version and name entries
### Rationale Automates the testcases using the npm test infrastructure (`npm test`). Adds the actual expected outcomes as defined files which are verified to be the same Also ensures the output file has a EOF character at end.
### Rationale Tests were not providing a bad exit code & the latest npm v7.13.0, also no longer throws an abnoxous error on a failed run-scripts. This should be a bit easier to determine when testing.
|
Looks great. Thanks! |
|
Published as 1.1.1 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
@elpete, thanks again for making this fork (fixing my mgibeau#8) and publishing it on npm since the original seems to be dormant.
I figured I can return the favor a bit and made some updates to your package. Primarily small simple edits in this PR.
Purpose
Update this package so that it works properly and does not get mixed up with the original gitlab-npm-audit-parser when using
npx. Update the readme to adequately define this package as well as automate the tests for the project to ensure the output is the desired output all the time.How to verify