Closed akash2144 closed 9 months ago
PR contains some changes from PR #696. Once that PR is merged will rebase this PR.
Most of the codacy issues are in conflict , One of them should be disabled. Ex. D203: 1 blank line required before class docstring D211: No blank lines allowed before class docstring
Do we have separate ticket for covering unittests? @akash2144
@mariyappanp Currently no API is available for testing, Once API is available will write test case there. CORTX-30029
It would be better to maintain rest api framework if we separate it from ha service like ha/framework/restapi. What do you think, @akash2144 @shaileshsvc @ArchanaLimaye
This issue/pull request has been marked as needs attention
as it has been left pending without new activity for 4 days. Tagging @shaileshsvc @ArchanaLimaye for appropriate assignment. Sorry for the delay & Thank you for contributing to CORTX. We will get back to you as soon as possible.
This issue/pull request has been marked as needs attention
as it has been left pending without new activity for 4 days. Tagging @shaileshsvc @ArchanaLimaye for appropriate assignment. Sorry for the delay & Thank you for contributing to CORTX. We will get back to you as soon as possible.
Is this PR still relevant? If not, can we close it?
This issue/pull request has been marked as needs attention
as it has been left pending without new activity for 4 days. Tagging @shaileshsvc @ArchanaLimaye for appropriate assignment. Sorry for the delay & Thank you for contributing to CORTX. We will get back to you as soon as possible.
Closing as an obsolete
Signed-off-by: Akash Dudhane akash.a.dudhane@seagate.com
Problem Statement
Design
Coding
Testing
Review Checklist
Review Checklist
Documentation
Checklist for Author