Skip to content

Support validation of multiple AIPs in an output#125

Merged
rohenaz merged 6 commits intomasterfrom
multiple-aip
Feb 14, 2025
Merged

Support validation of multiple AIPs in an output#125
rohenaz merged 6 commits intomasterfrom
multiple-aip

Conversation

@shruggr
Copy link
Collaborator

@shruggr shruggr commented Feb 14, 2025

No description provided.

@mergify
Copy link
Contributor

mergify bot commented Feb 14, 2025

Welcome to our open-source project! 💘

@codecov
Copy link

codecov bot commented Feb 14, 2025

Codecov Report

Attention: Patch coverage is 81.53846% with 12 lines in your changes missing coverage. Please review.

Project coverage is 55.73%. Comparing base (69d9dae) to head (8366080).
Report is 7 commits behind head on master.

Files with missing lines Patch % Lines
bob.go 81.53% 12 Missing ⚠️
Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #125      +/-   ##
==========================================
+ Coverage   50.00%   55.73%   +5.73%     
==========================================
  Files           5        5              
  Lines         270      305      +35     
==========================================
+ Hits          135      170      +35     
  Misses        125      125              
  Partials       10       10              
Flag Coverage Δ
unittests 55.73% <81.53%> (+5.73%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files with missing lines Coverage Δ
bob.go 58.79% <81.53%> (+9.81%) ⬆️

Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 69d9dae...8366080. Read the comment docs.

@rohenaz rohenaz self-requested a review February 14, 2025 17:31
Copy link
Member

@rohenaz rohenaz left a comment

Choose a reason for hiding this comment

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

lgtm

@rohenaz rohenaz merged commit 7925c2f into master Feb 14, 2025
6 checks passed
@mergify mergify bot deleted the multiple-aip branch February 14, 2025 17:34
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