Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Update README.md #6

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Update README.md #6

wants to merge 1 commit into from

Conversation

Namanrepo
Copy link

No description provided.

@oppiabotlocal
Copy link

oppiabotlocal bot commented Mar 30, 2021

Hi @Namanrepo, can you complete the following:

  1. The body of this PR is missing the overview section, please update it to include the overview.
  2. The body of this PR is missing the checklist section, please update it to include the checklist.
  3. The body of this PR is missing the proof that changes are correct section, please update it to include the section.
    Thanks!

Repository owner deleted a comment from oppiabotlocal bot Mar 30, 2021
@Naman-1234 Naman-1234 self-requested a review March 31, 2021 22:11
Naman-1234
Naman-1234 requested changes Mar 31, 2021
View reviewed changes
Copy link
Owner

@Naman-1234 Naman-1234 left a comment

Choose a reason for hiding this comment

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

Changes are requested.

@oppiabotlocal oppiabotlocal bot removed the PR: LGTM label Mar 31, 2021
@oppiabotlocal
Copy link

oppiabotlocal bot commented Mar 31, 2021

Hi, @Namanrepo, Removing LGTM Label since the changes are requested. Thanks!.

Naman-1234
Naman-1234 requested changes Mar 31, 2021
View reviewed changes
Copy link
Owner

@Naman-1234 Naman-1234 left a comment

Choose a reason for hiding this comment

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

Changes Requested.

Naman-1234
Naman-1234 requested changes Mar 31, 2021
View reviewed changes
Copy link
Owner

@Naman-1234 Naman-1234 left a comment

Choose a reason for hiding this comment

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

Changes Requested.

Naman-1234
Naman-1234 approved these changes Mar 31, 2021
View reviewed changes
Copy link
Owner

@Naman-1234 Naman-1234 left a comment

Choose a reason for hiding this comment

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

hey

Naman-1234
Naman-1234 requested changes Mar 31, 2021
View reviewed changes
Copy link
Owner

@Naman-1234 Naman-1234 left a comment

Choose a reason for hiding this comment

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

Request Changes.

@Naman-1234 Naman-1234 self-requested a review April 5, 2021 17:13
@oppiabotlocal
Copy link

oppiabotlocal bot commented Apr 5, 2021

Hi @Namanrepo, the build of this PR is stale and this could result in tests failing in develop. Please update this pull request with the latest changes from develop. Thanks!

Naman-1234
Naman-1234 requested changes Apr 6, 2021
View reviewed changes
Copy link
Owner

@Naman-1234 Naman-1234 left a comment

Choose a reason for hiding this comment

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

Perform Changes.

@Naman-1234 Naman-1234 self-requested a review April 6, 2021 19:04
Naman-1234
Naman-1234 requested changes Apr 6, 2021
View reviewed changes
Copy link
Owner

@Naman-1234 Naman-1234 left a comment

Choose a reason for hiding this comment

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

Perform Changes.

@oppiabotlocal oppiabotlocal bot removed the PR: LGTM label Apr 6, 2021
@oppiabotlocal
Copy link

oppiabotlocal bot commented Apr 6, 2021

Hi, @Namanrepo, the LGTM Label was removed because the changes are requested on this PR. Thanks!.

@Naman-1234 Naman-1234 self-requested a review April 6, 2021 19:25
Naman-1234
Naman-1234 requested changes Apr 6, 2021
View reviewed changes
Copy link
Owner

@Naman-1234 Naman-1234 left a comment

Choose a reason for hiding this comment

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

Perform Changes.

@oppiabotlocal oppiabotlocal bot removed the PR: LGTM label Apr 6, 2021
Repository owner deleted a comment from oppiabotlocal bot Apr 6, 2021
@Naman-1234 Naman-1234 self-requested a review April 6, 2021 19:31
Naman-1234
Naman-1234 requested changes Apr 6, 2021
View reviewed changes
Copy link
Owner

@Naman-1234 Naman-1234 left a comment

Choose a reason for hiding this comment

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

changes

@oppiabotlocal oppiabotlocal bot removed the PR: LGTM label Apr 6, 2021
@oppiabotlocal
Copy link

oppiabotlocal bot commented Apr 6, 2021

Hi, @Namanrepo, the LGTM Label was removed because the changes are requested on this PR. Thanks!.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Naman-1234 Naman-1234 Naman-1234 requested changes

Assignees

@Namanrepo Namanrepo

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@Namanrepo @Naman-1234