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

Update SUPPORT.md#1456

Merged
johnbent merged 2 commits intomainfrom
novium258-patch-1
Mar 28, 2022
Merged

Update SUPPORT.md#1456
johnbent merged 2 commits intomainfrom
novium258-patch-1

Conversation

@novium258
Copy link
Copy Markdown
Contributor

@novium258 novium258 commented Mar 3, 2022 •
edited by markdownify Bot
Loading

removing office hours.
Signed-off-by: Rachel N rachel.novak@seagate.com

Describe your changes in brief

Changes

  • Why is this change required? What problem does it solve?
  • If proposing a new change then please raise an issue first

How Has This Been Tested? (Optional)

  • Please describe in detail how you tested your changes.
  • Include details of your testing environment, and the tests you ran to
  • How your change affects other areas of the code, etc.

Screenshots (if appropriate)

Checklist

  • tested locally
  • added new dependencies
  • updated the docs
  • added a test

View rendered SUPPORT.md

@novium258
Update SUPPORT.md
16ed4cb 
removing office hours. 
Signed-off-by: Rachel N <rachel.novak@seagate.com>
@stale
Copy link
Copy Markdown

stale Bot commented Mar 10, 2022

This issue/pull request has been marked as needs attention as it has been left pending without new activity for 4 days. Tagging @mukul-seagate11 for appropriate assignment. Sorry for the delay & Thank you for contributing to CORTX. We will get back to you as soon as possible.

@r-wambui
Copy link
Copy Markdown
Contributor

r-wambui commented Mar 15, 2022

@mukul-seagate11 @shailesh-vaidya Please review and merge.

@stale stale Bot removed the needs-attention label Mar 15, 2022
mukul-seagate11
mukul-seagate11 suggested changes Mar 16, 2022
View reviewed changes
Copy link
Copy Markdown
Contributor

@mukul-seagate11 mukul-seagate11 left a comment

Choose a reason for hiding this comment

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

@novium258, update your fork branch

@stale
Copy link
Copy Markdown

stale Bot commented Mar 20, 2022

This issue/pull request has been marked as needs attention as it has been left pending without new activity for 4 days. Tagging @mukul-seagate11 for appropriate assignment. Sorry for the delay & Thank you for contributing to CORTX. We will get back to you as soon as possible.

@cla-bot cla-bot Bot added the cla-signed label Mar 28, 2022
@stale stale Bot removed the needs-attention label Mar 28, 2022
@johnbent johnbent merged commit 6d94855 into main Mar 28, 2022
@johnbent johnbent deleted the novium258-patch-1 branch March 28, 2022 18:55
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

@shailesh-vaidya shailesh-vaidya Awaiting requested review from shailesh-vaidya shailesh-vaidya is a code owner

2 more reviewers

@mukul-seagate11 mukul-seagate11 mukul-seagate11 requested changes

@johnbent johnbent johnbent approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

cla-signed

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@novium258 @r-wambui @johnbent @mukul-seagate11