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

Updated Codacy Badge #720

Closed
wants to merge 2 commits into from
Closed

Updated Codacy Badge #720

wants to merge 2 commits into from

Conversation

hessio
Copy link

@hessio hessio commented Aug 8, 2022

Resolves issue #1528

Signed-off-by: hessio patric.hession@seagate.com

Signed-off-by: Patrick Hession patrick.hession@seagate.com

Problem Statement

  • Problem statement

Design

  • For Bug, Describe the fix here.
  • For Feature, Post the link for design

Coding

  • Coding conventions are followed and code is consistent

Testing

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

Review Checklist

  • PR is self reviewed
  • JIRA number/GitHub Issue added to PR
  • Jira and state/status is updated and JIRA is updated with PR link
  • Check if the description is clear and explained
  • Is there a change in filename/package/module or signature? [Y/N]:
  • If yes for above point, is a notification sent to all other cortx components? [Y/N]
  • Side effects on other features (deployment/upgrade)? [Y/N]
  • Dependencies on other component(s)? [Y/N]
  • If yes for above point, post link to the corresponding PR.
    

Review Checklist

  • Is perfline test run and the report with and without the changes updated in the PR? [Y/N]:

Documentation

Checklist for Author

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

View rendered README.md

Sorry, something went wrong.

Resolves (issue #1528)[Seagate/cortx#1528]

Signed-off-by: hessio <patric.hession@seagate.com>

Signed-off-by: Patrick Hession <patrick.hession@seagate.com>
@welcome
Copy link

welcome bot commented Aug 8, 2022

Thanks for your contribution in opening this pull request! Now you can be rewarded with a CORTX sticker by requesting cortx sticker
In the meantime, please check out the contributing guidelines and explore other ways you can get involved.

@stale
Copy link

stale bot commented Aug 13, 2022

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.

@stale
Copy link

stale bot commented Sep 9, 2022

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.

@stale stale bot added the needs-attention label Sep 9, 2022
@hessio
Copy link
Author

hessio commented Sep 12, 2022

Codacy Badge link was broken so I changed the link to the new and working link from the Codacy dashboard.

Kindly review when you have time: @ArchanaLimaye @mssawant

@shailesh-vaidya
Copy link
Contributor

Closing as an obsolete

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Codacy Badge is not rendering properly for five docs
5 participants