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

Final Submission: Open Source contribution to better-npm-audit #2030

Closed
wants to merge 15 commits into from
Closed

Final Submission: Open Source contribution to better-npm-audit #2030

wants to merge 15 commits into from

Conversation

bencivjan
Copy link

Assignment Proposal

Title

Contributing to better-npm-audit

Names and KTH ID

Deadline

Deadline task 5

Category

Contribution to open-source

Description

better-npm-audit is a repository for creating a more informative npm-audit output. This relates to DevOps because it is an improvement on an existing package manager and is used to make software development more efficient to debug and automates common development tasks. It also relates to DevOps because it makes it easier to understand dependencies vulnerabilities and possible security failures.

We have addressed an issue that was raised and requested to be implemented by the author of the library. We will create a feature that splits the output of better-npm-audit into 2 tables, separated as developer dependencies and production dependencies. This will help developers prioritize which packages to fix, since a developer vulnerability is generally not as problematic.

The issue that we will be addressing and our dialogue with the author are located at: jeemok/better-npm-audit#67.

The repository has 77 stars, 218 commits, and has an active community on GitHub since we recently had a conversation with the author.

Submission:
The final pr can be found here: jeemok/better-npm-audit#78.

We have implemented the feature and passed all tests, including tests we modified to cover the new feature. We are currently awaiting approval from the author.

@dd2482-bot
Copy link
Collaborator

Student ttle already registered for this task

If not from your group, fetch the upstream.

This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants