Skip to content

test - #18

Open
bstoll wants to merge 8 commits into
mainfrom
hines-fork
Open

test#18
bstoll wants to merge 8 commits into
mainfrom
hines-fork

Conversation

@bstoll

@bstoll bstoll commented Jul 22, 2026

Copy link
Copy Markdown
Owner

No description provided.

anh-nguyen-ciena and others added 7 commits June 25, 2026 22:14
* Modified proto/topo/topo.proto
  * add Ciena vendor and ciena_saos type
* Modified proto/topo/topo.pb.go
  * changes generated by topo.proto
* Modified topo/topo.go
  * add ciena node into import
* Added topo/node/ciena/ciena.go
  * ciena node implementation
* Added proto/ciena.proto
  * add Ciena proto - define Ciena specific vendor data for KNE
* Added proto/ciena/ciena.pb.go
  * file generated by proto/ciena.proto
@coveralls

coveralls commented Jul 22, 2026

Copy link
Copy Markdown

Coverage Report for CI Build 29898457334

Coverage increased (+0.5%) to 42.133%

Details

  • Coverage increased (+0.5%) from the base build.
  • Patch coverage: 134 uncovered changes across 2 files (203 of 337 lines covered, 60.24%).
  • 1 coverage regression across 1 file.

Uncovered Changes

File Changed Covered %
proto/ciena/ciena.pb.go 110 0 0.0%
topo/node/ciena/ciena.go 227 203 89.43%

Coverage Regressions

1 previously-covered line in 1 file lost coverage.

File Lines Losing Coverage Coverage
topo/node/arista/arista.go 1 67.26%

Coverage Stats

Coverage Status
Relevant Lines: 12819
Covered Lines: 5401
Line Coverage: 42.13%
Coverage Strength: 0.48 hits per line

💛 - Coveralls

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.

4 participants