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

Conversation

@pratik-patil-5611
Copy link
Contributor

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

Updating following files for formatting issues
doc/trace.md
fdmi/plugins/motr-fdmi-app.md
doc/ADDB_Monitoring.md
doc/Seagate-FDMI-Design-Notes.md
doc/End-to-end-Data-Integrity.md
hsm/README.md
scripts/provisioning/README.md
doc/workarounds.md
doc/outdated/Containers.md

Problem Statement

[Codacy code cleanup] occurrence: Warn when the spacing between a list item’s bullet and its content violates.

Design

Fixed formatting issues

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/ADDB_Monitoring.md
View rendered doc/End-to-end-Data-Integrity.md
View rendered doc/Seagate-FDMI-Design-Notes.md
View rendered doc/outdated/Containers.md
View rendered doc/trace.md
View rendered doc/workarounds.md
View rendered fdmi/plugins/motr-fdmi-app.md
View rendered hsm/README.md
View rendered scripts/provisioning/README.md

@cla-bot
Copy link

cla-bot bot commented Aug 1, 2022

Thank you for your pull request and welcome to our community. We could not parse the GitHub identity of the following contributors: root, Pratik Patil.
This is most likely caused by a git client misconfiguration; please make sure to:

  1. check if your git client is configured with an email to sign commits git config --list | grep email
  2. If not, set it up using git config --global user.email [email protected]
  3. Make sure that the git commit email is configured in your GitHub account settings, see https://github.com/settings/emails

@cla-bot
Copy link

cla-bot bot commented Aug 1, 2022

Thank you for your pull request and welcome to our community. We could not parse the GitHub identity of the following contributors: root, Pratik Patil.
This is most likely caused by a git client misconfiguration; please make sure to:

  1. check if your git client is configured with an email to sign commits git config --list | grep email
  2. If not, set it up using git config --global user.email [email protected]
  3. Make sure that the git commit email is configured in your GitHub account settings, see https://github.com/settings/emails

2 similar comments
@cla-bot
Copy link

cla-bot bot commented Aug 1, 2022

Thank you for your pull request and welcome to our community. We could not parse the GitHub identity of the following contributors: root, Pratik Patil.
This is most likely caused by a git client misconfiguration; please make sure to:

  1. check if your git client is configured with an email to sign commits git config --list | grep email
  2. If not, set it up using git config --global user.email [email protected]
  3. Make sure that the git commit email is configured in your GitHub account settings, see https://github.com/settings/emails

@cla-bot
Copy link

cla-bot bot commented Aug 1, 2022

Thank you for your pull request and welcome to our community. We could not parse the GitHub identity of the following contributors: root, Pratik Patil.
This is most likely caused by a git client misconfiguration; please make sure to:

  1. check if your git client is configured with an email to sign commits git config --list | grep email
  2. If not, set it up using git config --global user.email [email protected]
  3. Make sure that the git commit email is configured in your GitHub account settings, see https://github.com/settings/emails

@cortx-admin
Copy link

Can one of the admins verify this patch?

1. README.md
2. doc/ADDB.md
3. doc/CORTX_MOTR_ARCHITECTURE.md
4. doc/DTM.md
5. doc/HDMI-High-Level-Decomposition.md
6. doc/FOPFOM-Programming-Guide.md
7. doc/Running_Motr_Across_a_Cluster.md
8. doc/Seagate-FDMI-Design-Notes.md
9. doc/Seagate-FDMI-HLD.md

Signed-off-by: Pratik Patil <[email protected]>
1. README.md
2. doc/ADDB.md
3. doc/CORTX_MOTR_ARCHITECTURE.md
4. doc/DTM.md
5. doc/HDMI-High-Level-Decomposition.md
6. doc/FOPFOM-Programming-Guide.md
7. doc/Running_Motr_Across_a_Cluster.md
8. doc/Seagate-FDMI-Design-Notes.md
9. doc/Seagate-FDMI-HLD.md

Signed-off-by: Pratik Patil <[email protected]>
Signed-off-by: Pratik Patil <[email protected]>
Signed-off-by: Pratik Patil <[email protected]>
@cla-bot
Copy link

cla-bot bot commented Aug 2, 2022

Thank you for your pull request and welcome to our community. We could not parse the GitHub identity of the following contributors: Pratik Patil.
This is most likely caused by a git client misconfiguration; please make sure to:

  1. check if your git client is configured with an email to sign commits git config --list | grep email
  2. If not, set it up using git config --global user.email [email protected]
  3. Make sure that the git commit email is configured in your GitHub account settings, see https://github.com/settings/emails

1 similar comment
@cla-bot
Copy link

cla-bot bot commented Aug 2, 2022

Thank you for your pull request and welcome to our community. We could not parse the GitHub identity of the following contributors: Pratik Patil.
This is most likely caused by a git client misconfiguration; please make sure to:

  1. check if your git client is configured with an email to sign commits git config --list | grep email
  2. If not, set it up using git config --global user.email [email protected]
  3. Make sure that the git commit email is configured in your GitHub account settings, see https://github.com/settings/emails

@cla-bot
Copy link

cla-bot bot commented Aug 2, 2022

Thank you for your pull request and welcome to our community. We could not parse the GitHub identity of the following contributors: Pratik Patil.
This is most likely caused by a git client misconfiguration; please make sure to:

  1. check if your git client is configured with an email to sign commits git config --list | grep email
  2. If not, set it up using git config --global user.email [email protected]
  3. Make sure that the git commit email is configured in your GitHub account settings, see https://github.com/settings/emails

Signed-off-by: Pratik Kumar Patil <[email protected]>
@cla-bot
Copy link

cla-bot bot commented Aug 2, 2022

Thank you for your pull request and welcome to our community. We could not parse the GitHub identity of the following contributors: Pratik Patil.
This is most likely caused by a git client misconfiguration; please make sure to:

  1. check if your git client is configured with an email to sign commits git config --list | grep email
  2. If not, set it up using git config --global user.email [email protected]
  3. Make sure that the git commit email is configured in your GitHub account settings, see https://github.com/settings/emails

Signed-off-by: Pratik Patil <[email protected]>
@cla-bot
Copy link

cla-bot bot commented Aug 4, 2022

Thank you for your pull request and welcome to our community. We could not parse the GitHub identity of the following contributors: Pratik Patil.
This is most likely caused by a git client misconfiguration; please make sure to:

  1. check if your git client is configured with an email to sign commits git config --list | grep email
  2. If not, set it up using git config --global user.email [email protected]
  3. Make sure that the git commit email is configured in your GitHub account settings, see https://github.com/settings/emails

@rkothiya
Copy link
Contributor

rkothiya commented Aug 4, 2022

giving an exception for pre-merge jobs as the changes are related to documentation.

@rkothiya rkothiya added the Exception Allowed to merge without runing ut and st label Aug 4, 2022
@rkothiya rkothiya merged commit 1f79f11 into Seagate:main Aug 4, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Exception Allowed to merge without runing ut and st

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants