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

feature: Add support for including specific columns in audit report #100

Merged
merged 3 commits into from
Sep 3, 2024

Conversation

GauravChinavle
Copy link
Contributor

  • Changed column name from "Sev." to "Severity"
  • Implemented the -i flag to specify columns to include in the audit report
  • Now, users can choose to include selected columns like Module, Title, and Severity by using the -i option
  • Example usage: better-npm-audit audit -i Module,Title,Severity

- Implemented the `-i` flag to specify columns to include in the audit report
- Now, users can choose to include selected columns like Module, Title, and Severity by using the `-i` option
- Example usage: `better-npm-audit audit -i Module,Title,Severity`
@jeemok
Copy link
Owner

jeemok commented Sep 1, 2024

Thank you for your contribution, @GauravChinavle ! :)

PR looks good to me, can you also include the new flag -i into README.md please?

package.json Outdated Show resolved Hide resolved
Gaurav Chinawale and others added 2 commits September 3, 2024 11:22
- Added `-i` flag details to README.md
- Retained the `prebuild` command
- Performed `npm audit fix` to address vulnerabilities
@GauravChinavle
Copy link
Contributor Author

chore: apply review changes

  • Added -i flag details to README.md
  • Retained the `prebuild` command
  • Performed npm audit fix to address vulnerabilities

@jeemok
Copy link
Owner

jeemok commented Sep 3, 2024

thank you @GauravChinavle !

@jeemok jeemok merged commit a414647 into jeemok:master Sep 3, 2024
5 checks passed
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