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

Update MSBM w/ new features in Tangle (same branch name) #50

Merged
merged 12 commits into from
Feb 21, 2025

Conversation

drewstone
Copy link
Contributor

No description provided.

Copy link

github-actions bot commented Jan 22, 2025

LCOV of commit 5de7c3a during Foundry CI #58

Summary coverage rate:
  lines......: 78.8% (294 of 373 lines)
  functions..: 73.9% (82 of 111 functions)
  branches...: 62.2% (23 of 37 branches)

Files changed coverage rate:
                                       |Lines       |Functions  |Branches    
  Filename                             |Rate     Num|Rate    Num|Rate     Num
  ===========================================================================
  src/BlueprintServiceManagerBase.sol  |45.0%     60|12.0%    25|    -      0
  src/MasterBlueprintServiceManager.sol|19.9%    136|40.7%    27|    -      0
  src/Permissions.sol                  |62.5%      8| 0.0%     3|    -      0
  test/HookTest.sol                    |    -      0|    -     0|    -      0
  test/MockBlueprintServiceManager.sol |55.6%     18| 0.0%     9|    -      0

⛔ The code coverage is too low: 78.82. Expected at least 85.

@drewstone drewstone marked this pull request as ready for review February 6, 2025 07:04
@shekohex
Copy link
Contributor

@drewstone you will need to add more testing here to increase the coverage so we can merge this and publish a new version of tnt-core.

@drewstone drewstone merged commit 8f4975b into main Feb 21, 2025
1 of 2 checks passed
@Serial-ATA Serial-ATA deleted the drew/slashing-updates branch February 21, 2025 16: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