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

Add Maven property for Node Audit Analyzer URL #5143

Merged
merged 1 commit into from
Dec 15, 2022

Conversation

A-Fitz-Nelnet
Copy link
Contributor

Description of Change

  • Added the Node Audit Analyzer URL property to the Maven configuration.
  • Fixed Javadoc for the nodePackageSkipDevDependencies Maven property that incorrectly referred to it as a property for the Node Audit Analyzer instead of the Node.js analyzer.

Have test cases been added to cover the new functionality?

No, feel free to point to where I should do so if necessary.

@boring-cyborg boring-cyborg bot added documentation site documentation maven changes to the maven plugin labels Dec 12, 2022
@A-Fitz-Nelnet
Copy link
Contributor Author

I did not find it necessary to create an issue since the change is fairly simple. Please let me know if I misunderstood the Contribution Guidelines for the project.

@jeremylong jeremylong added this to the 7.4.2 milestone Dec 13, 2022
@A-Fitz-Nelnet
Copy link
Contributor Author

Thanks for the approval @jeremylong. My coworkers and I appreciate all of your hard work on this project!

@jeremylong jeremylong merged commit 5435749 into jeremylong:main Dec 15, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation site documentation maven changes to the maven plugin
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants