Skip to content
This repository has been archived by the owner on May 3, 2024. It is now read-only.

CORTX-33978- codacy cleanup: Fixed codacy formatting issues #2087

Merged
merged 1 commit into from
Aug 18, 2022

Conversation

pratik-patil-5611
Copy link
Contributor

@pratik-patil-5611 pratik-patil-5611 commented Aug 18, 2022

Signed-off-by: Pratik Patil pratik.k.patil@seagate.com

  • Codacy warnings in below given files:
    • [fenced-code-flag]
      • doc/Motr-Lnet-Transport.md
      • doc/Running_Motr_Across_a_Cluster.md
      • doc/FOPFOM-Programming-Guide.md
      • doc/HLD-OF-Motr-LNet-Transport.md
      • doc/HLD-of-Catalogue-Service.md
      • doc/motr-object-app.md

Design

  • [fenced-code-flag] -> added name of programming language after 3 backquotes ```

Coding

Checklist for Author

  • Coding conventions are followed and code is consistent

Testing

Checklist for Author

  • Unit and System Tests are added
  • Test Cases cover Happy Path, Non-Happy Path and Scalability
  • Testing was performed with RPM

Impact Analysis

Checklist for Author/Reviewer/GateKeeper

  • Interface change (if any) are documented
  • Side effects on other features (deployment/upgrade)
  • Dependencies on other component(s)

Review Checklist

Checklist for Author

  • JIRA number/GitHub Issue added to PR
  • PR is self reviewed
  • Jira and state/status is updated and JIRA is updated with PR link
  • Check if the description is clear and explained

Documentation

Checklist for Author

  • Changes done to WIKI / Confluence page / Quick Start Guide

View rendered doc/FOPFOM-Programming-Guide.md
View rendered doc/HLD-OF-Motr-LNet-Transport.md
View rendered doc/HLD-of-Catalogue-Service.md
View rendered doc/Motr-Lnet-Transport.md
View rendered doc/Running_Motr_Across_a_Cluster.md
View rendered doc/motr-object-app.md

Signed-off-by: Pratik Patil <pratik.k.patil@seagate.com>
@rkothiya
Copy link
Contributor

Can one of the admins verify this patch?

@rkothiya
Copy link
Contributor

rkothiya commented Aug 18, 2022

Giving exception for pre-merge and ci/cd as it is related to documentation.

@rkothiya rkothiya merged commit 9935bf1 into Seagate:main Aug 18, 2022
kiwionly2 pushed a commit to kiwionly2/cortx-motr that referenced this pull request Aug 30, 2022
[fenced-code-flag] -> added name of programming language after 3 backquotes ```

Signed-off-by: Pratik Patil <pratik.k.patil@seagate.com>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants