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.

Sign up for GitHub

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

Jump to bottom

RT-1535:(charts-on-fhir) npm audit #258

Merged
merged 3 commits into from
Sep 20, 2024
Merged

RT-1535:(charts-on-fhir) npm audit #258

merged 3 commits into from
Sep 20, 2024

Conversation

vdeshkar
Copy link
Contributor

Overview

  • Ran npm audit.
  • Fixed audit issues.

How it was tested

  • Tested by launching charts-on-fhir apps

Checklist

  • The title contains a short meaningful summary
  • I have added a link to this PR in the Jira issue
  • I have described how this was tested
  • I have included screen shots for changes that affect the user interface
  • I have updated unit tests
  • I have run unit tests locally
  • I have updated documentation (including README)

semicolin
semicolin reviewed Sep 16, 2024
View reviewed changes
@vdeshkar vdeshkar requested a review from semicolin September 16, 2024 13:56
@sonarqubecloud SonarQubeCloud
Copy link

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarCloud

@vdeshkar vdeshkar merged commit 16e537f into main Sep 20, 2024
5 checks passed
@vdeshkar vdeshkar deleted the RT-1535-sapphire-npm-audit-charts-on-fhir branch September 20, 2024 08:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@semicolin semicolin semicolin approved these changes

@shubhamparikh927 shubhamparikh927 shubhamparikh927 approved these changes

@yashpmohod yashpmohod yashpmohod approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@vdeshkar @semicolin @shubhamparikh927 @yashpmohod