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

[packchk] no error message if ../etc/PACK.xsd not found #786 (#576) #946

Merged
merged 1 commit into from
May 30, 2023

Conversation

grasci-arm
Copy link
Collaborator

added:

  • Schema Validation Test

changed:

  • Xerxes continues checking after first error (do not treat everything as critical)
  • added test cases for schema check

…ck#786 (Open-CMSIS-Pack#576)

added:
- Schema Validation Test

changed:
- Xerxes continues checking after first error (do not treat everything as critical)
- added test cases for schema check
@codecov
Copy link

codecov bot commented May 30, 2023

Codecov Report

Merging #946 (525631e) into main (73dda22) will increase coverage by 0.04%.
The diff coverage is 100.00%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #946      +/-   ##
==========================================
+ Coverage   53.85%   53.89%   +0.04%     
==========================================
  Files         116      116              
  Lines       22339    22339              
  Branches    12360    12359       -1     
==========================================
+ Hits        12030    12040      +10     
+ Misses       8194     8188       -6     
+ Partials     2115     2111       -4     
Flag Coverage Δ
packchk-cov 60.97% <100.00%> (+0.36%) ⬆️
svdconv-cov 32.56% <ø> (ø)

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

Impacted Files Coverage Δ
tools/packchk/src/CreateModel.cpp 40.84% <ø> (+1.95%) ⬆️
tools/packchk/src/PackOptions.cpp 58.73% <100.00%> (+1.81%) ⬆️

... and 2 files with indirect coverage changes

@github-actions
Copy link

github-actions bot commented May 30, 2023

Test Results

  30 files    54 suites   4s ⏱️
122 tests 122 ✔️ 0 💤 0
366 runs  366 ✔️ 0 💤 0

Results for commit 525631e.

♻️ This comment has been updated with latest results.

@edriouk edriouk merged commit 200189a into Open-CMSIS-Pack:main May 30, 2023
@edriouk edriouk deleted the outbound/git#786 branch May 30, 2023 08:38
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.

3 participants