Skip to content

mark link error when port.nni is removed (data model dependency) - #303

Merged
YufengXin merged 13 commits into
mainfrom
302-port-nni-removed
Apr 24, 2026
Merged

mark link error when port.nni is removed (data model dependency)#303
YufengXin merged 13 commits into
mainfrom
302-port-nni-removed

Conversation

@YufengXin

@YufengXin YufengXin commented Feb 3, 2026

Copy link
Copy Markdown
Collaborator

@YufengXin YufengXin self-assigned this Feb 3, 2026
@YufengXin YufengXin linked an issue Feb 3, 2026 that may be closed by this pull request
@YufengXin
YufengXin marked this pull request as draft February 3, 2026 22:23
@coveralls

coveralls commented Feb 3, 2026

Copy link
Copy Markdown

Coverage Report for CI Build 24258182759

Coverage decreased (-0.6%) to 90.332%

Details

  • Coverage decreased (-0.6%) from the base build.
  • Patch coverage: 12 uncovered changes across 1 file (11 of 23 lines covered, 47.83%).
  • No coverage regressions found.

Uncovered Changes

File Changed Covered %
src/sdx_pce/topology/manager.py 23 11 47.83%

Coverage Regressions

No coverage regressions found.


Coverage Stats

Coverage Status
Relevant Lines: 1590
Covered Lines: 1458
Line Coverage: 91.7%
Relevant Branches: 758
Covered Branches: 663
Branch Coverage: 87.47%
Branches in Coverage %: Yes
Coverage Strength: 1.83 hits per line

💛 - Coveralls

@YufengXin YufengXin changed the title make link down when port.nni is removed mark link error when port.nni is removed Mar 30, 2026
@YufengXin
YufengXin requested a review from congwang09 March 30, 2026 21:31
@YufengXin YufengXin changed the title mark link error when port.nni is removed mark link error when port.nni is removed (data model dependency) Mar 30, 2026
@YufengXin
YufengXin merged commit 64c1c31 into main Apr 24, 2026
4 checks passed
@YufengXin
YufengXin deleted the 302-port-nni-removed branch April 24, 2026 14:31
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.

Port nni removed

3 participants