Skip to content

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

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

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

Conversation

@YufengXin
Copy link
Copy Markdown
Collaborator

@YufengXin YufengXin commented Feb 3, 2026

@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
Copy link
Copy Markdown

coveralls commented Feb 3, 2026

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