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

CORTX-33905: Update CODEOWNERS #6401

Merged
merged 5 commits into from
Aug 18, 2022
Merged

CORTX-33905: Update CODEOWNERS #6401

merged 5 commits into from
Aug 18, 2022

Conversation

pankajgborole
Copy link
Contributor

Provided access to Jai

Signed-off-by: pankajgborole pankaj.g.borole@seagate.com

Problem Statement

Add Jaikumar Gidwani as code owner for cortx-prvsnr repo

Design

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

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

Provided access to Jai

Signed-off-by: pankajgborole <pankaj.g.borole@seagate.com>
@welcome
Copy link

welcome bot commented Aug 10, 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.

pankajgborole and others added 2 commits August 10, 2022 16:25
Copy link
Contributor

@nitin-seagate nitin-seagate left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

CODEOWNERS Outdated Show resolved Hide resolved
@hessio hessio added the Status: Changes Requested reviewer has requested some changes to the PR label Aug 16, 2022
Signed-off-by: pankajgborole <pankaj.g.borole@seagate.com>
@pankajgborole pankajgborole removed the Status: Changes Requested reviewer has requested some changes to the PR label Aug 18, 2022
@tanujashinde0405 tanujashinde0405 merged commit 6d91a2d into main Aug 18, 2022
@welcome
Copy link

welcome bot commented Aug 18, 2022

Thanks for your contribution to CORTX! 🎉

@tanujashinde0405 tanujashinde0405 deleted the Pankaj-CORTX-33905 branch August 18, 2022 06:10
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.

4 participants