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

[projmgr] Add for-context and for-compiler spelling check #1121

Conversation

grasci-arm
Copy link
Collaborator

@grasci-arm grasci-arm commented Sep 11, 2023

Address #926

@codecov
Copy link

codecov bot commented Sep 11, 2023

Codecov Report

Merging #1121 (cd10bc3) into main (24664a1) will increase coverage by 0.03%.
Report is 1 commits behind head on main.
The diff coverage is 74.50%.

@@            Coverage Diff             @@
##             main    #1121      +/-   ##
==========================================
+ Coverage   58.09%   58.13%   +0.03%     
==========================================
  Files         116      116              
  Lines       22833    22884      +51     
  Branches    12683    12724      +41     
==========================================
+ Hits        13265    13303      +38     
- Misses       7338     7342       +4     
- Partials     2230     2239       +9     
Flag Coverage Δ
projmgr-cov 82.68% <74.50%> (-0.09%) ⬇️

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

Files Changed Coverage
tools/projmgr/src/ProjMgrWorker.cpp 73.46%
tools/projmgr/src/ProjMgr.cpp 100.00%

@github-actions
Copy link

Test Results

    3 files    15 suites   2m 20s ⏱️
270 tests 270 ✔️ 0 💤 0
810 runs  810 ✔️ 0 💤 0

Results for commit cd10bc3.

@edriouk edriouk merged commit f14e09a into Open-CMSIS-Pack:main Sep 12, 2023
32 of 33 checks passed
@edriouk edriouk deleted the outbound/projmgr-filters-spelling-check branch September 12, 2023 07: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.

3 participants