Closed zhang-ivy closed 1 year ago
I caught some issues with the comments in the RESTCapableHybridTopologyFactory. I cleaned up the comments in this PR. I also removed a redundant type check in get_rest_identifer -- the type is being checked twice, so I removed one of them.
get_rest_identifer
Resolves #???
Merging #1189 (9287ce3) into 0.10.x (186375d) will decrease coverage by 2.67%. The diff coverage is 83.33%.
2.67%
83.33%
codecov[bot]
commented
1 year ago
Description
I caught some issues with the comments in the RESTCapableHybridTopologyFactory. I cleaned up the comments in this PR. I also removed a redundant type check in
get_rest_identifer-- the type is being checked twice, so I removed one of them.Motivation and context
Resolves #???
How has this been tested?
Change log