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

Update packages #2024

Merged
merged 2 commits into from
Jan 23, 2024
Merged

Update packages #2024

merged 2 commits into from
Jan 23, 2024

Conversation

awharn
Copy link
Member

@awharn awharn commented Jan 22, 2024

What It Does
Updates jsonschema and mustache dependencies

How to Test

Review Checklist
I certify that I have:

Additional Comments

@awharn
Update packages
709f8bf 
Signed-off-by: Andrew W. Harn <[email protected]>
@codecov Codecov
Copy link

codecov bot commented Jan 22, 2024 •
edited
Loading

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (6fef2d5) 91.03% compared to head (cc9f4eb) 91.02%.

Additional details and impacted files 
@@            Coverage Diff             @@
##           master    #2024      +/-   ##
==========================================
- Coverage   91.03%   91.02%   -0.02%     
==========================================
  Files         636      636              
  Lines       18739    18739              
  Branches     3910     3910              
==========================================
- Hits        17059    17057       -2     
- Misses       1679     1681       +2     
  Partials        1        1

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

zFernand0
zFernand0 approved these changes Jan 22, 2024
View reviewed changes
Copy link
Member

@zFernand0 zFernand0 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! 😋

traeok
traeok approved these changes Jan 22, 2024
View reviewed changes
Copy link
Member

@traeok traeok left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM - thanks @awharn 👍

t1m0thyj
t1m0thyj approved these changes Jan 22, 2024
View reviewed changes
Copy link
Member

@t1m0thyj t1m0thyj left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Always nice to see our shrinkwrap shrink 😋 Thanks for keeping our dependencies current @awharn!

@sonarcloud SonarCloud
Copy link

sonarcloud bot commented Jan 23, 2024

Quality Gate Passed Quality Gate passed

Kudos, no new issues were introduced!

0 New issues
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud

@awharn awharn merged commit f937576 into master Jan 23, 2024
19 checks passed
@awharn awharn deleted the update-dependencies branch January 23, 2024 15:39
@awharn awharn added the release-current Indicates that there is no new functionality being delivered label Jan 23, 2024
@adam-wolfe adam-wolfe mentioned this pull request Feb 27, 2024
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@KevinLoesch1 KevinLoesch1 KevinLoesch1 approved these changes

@traeok traeok traeok approved these changes

@zFernand0 zFernand0 zFernand0 approved these changes

@gejohnston gejohnston gejohnston approved these changes

@t1m0thyj t1m0thyj t1m0thyj approved these changes

@ATorrise ATorrise Awaiting requested review from ATorrise

Assignees

@awharn awharn

Labels
release-current Indicates that there is no new functionality being delivered
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
6 participants
@awharn @t1m0thyj @gejohnston @zFernand0 @traeok @KevinLoesch1