Skip to content

Conversation

@moliholy
Copy link
Collaborator

@moliholy moliholy commented Nov 4, 2025

As discussed in r0gue-io/base-parachain#69 (review), this PR removes the network.toml file from the base-parachain template.

@moliholy moliholy requested a review from AlexD10S November 4, 2025 14:59
@moliholy moliholy self-assigned this Nov 4, 2025
@moliholy moliholy force-pushed the feat/remove-base-parachain-network-config branch from 5762fa0 to 35a98e6 Compare November 4, 2025 15:07
@codecov
Copy link

codecov bot commented Nov 4, 2025

Codecov Report

❌ Patch coverage is 80.00000% with 3 lines in your changes missing coverage. Please review.
✅ Project coverage is 76.96%. Comparing base (cfe2df3) to head (5bbe283).
⚠️ Report is 1 commits behind head on main.

Files with missing lines Patch % Lines
crates/pop-chains/src/new_chain.rs 80.00% 2 Missing and 1 partial ⚠️
@@            Coverage Diff             @@
##             main     #711      +/-   ##
==========================================
+ Coverage   76.92%   76.96%   +0.04%     
==========================================
  Files         111      111              
  Lines       25240    25251      +11     
  Branches    25240    25251      +11     
==========================================
+ Hits        19415    19434      +19     
+ Misses       3767     3761       -6     
+ Partials     2058     2056       -2     
Files with missing lines Coverage Δ
crates/pop-chains/src/new_chain.rs 80.70% <80.00%> (+2.06%) ⬆️

... and 1 file with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@moliholy moliholy force-pushed the feat/remove-base-parachain-network-config branch from 35a98e6 to 7386dec Compare November 5, 2025 18:31
Copy link
Collaborator

@AlexD10S AlexD10S 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 for the 'workaround' checking the version

@moliholy moliholy merged commit 47b4803 into main Nov 6, 2025
19 checks passed
@moliholy moliholy deleted the feat/remove-base-parachain-network-config branch November 6, 2025 09:49
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