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.

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

PR by hatboysam-test #13

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

PR by hatboysam-test #13

wants to merge 1 commit into from

Conversation

alicethecoder
Copy link
Collaborator

@alicethecoder alicethecoder commented Aug 30, 2021

Quick description

@codeapprove
Copy link

codeapprove bot commented Aug 30, 2021

👋 Review this Pull Request on CodeApprove here:
https://codeapprove.com/pr/codeapprove/meta/13

@alicethecoder alicethecoder changed the title Update index.js PR by hatboysam-test Aug 30, 2021
@codeapprove-dev
Copy link

👋 Review this Pull Request on CodeApprove here:
http://localhost:8080/pr/codeapprove/meta/13

Copy link
Contributor

@hatboysam hatboysam left a comment

Choose a reason for hiding this comment

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

Automated comment from CodeApprove (go to review)

✔️ Approved

This looks good 👍


👀 @hatboysam-test it's your turn please take a look

Copy link
Contributor

@hatboysam hatboysam left a comment

Choose a reason for hiding this comment

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

Automated comment from CodeApprove (go to review)

Comment


👀 @hatboysam-test it's your turn please take a look

Copy link
Contributor

@hatboysam hatboysam left a comment

Choose a reason for hiding this comment

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

Automated comment from CodeApprove (go to review)

❌ Undo Approval

Undo approval

Copy link
Contributor

@hatboysam hatboysam left a comment

Choose a reason for hiding this comment

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

Automated comment from CodeApprove (go to review)

Comment

Copy link
Contributor

@hatboysam hatboysam left a comment

Choose a reason for hiding this comment

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

Automated comment from CodeApprove (go to review)

Add comment

Copy link
Contributor

@hatboysam hatboysam left a comment

Choose a reason for hiding this comment

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

Automated comment from CodeApprove (go to review)

Add a comment

Copy link
Contributor

@hatboysam hatboysam left a comment

Choose a reason for hiding this comment

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

Automated comment from CodeApprove (go to review)

Comment

Copy link
Contributor

@hatboysam hatboysam left a comment

Choose a reason for hiding this comment

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

Automated comment from CodeApprove (go to review)

Again

Copy link
Contributor

@hatboysam hatboysam left a comment

Choose a reason for hiding this comment

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

Automated comment from CodeApprove (go to review)

OK one more try

Copy link
Contributor

@hatboysam hatboysam left a comment

Choose a reason for hiding this comment

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

Automated comment from CodeApprove (go to review)

Again again

Copy link
Contributor

@hatboysam hatboysam left a comment

Choose a reason for hiding this comment

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

Automated comment from CodeApprove (go to review)

✔️ Approved

I approve


👀 @hatboysam-test it's your turn please take a look

Copy link
Contributor

@hatboysam hatboysam left a comment

Choose a reason for hiding this comment

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

Automated comment from CodeApprove (go to review)

Ok


👀 @hatboysam-test it's your turn please take a look

Copy link
Contributor

@hatboysam hatboysam left a comment

Choose a reason for hiding this comment

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

Automated comment from CodeApprove (go to review)

Comment


👀 @hatboysam-test it's your turn please take a look

Copy link
Contributor

@hatboysam hatboysam left a comment

Choose a reason for hiding this comment

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

Automated comment from CodeApprove (go to review)

✔️ Approved (1 unresolved comments)


In: Discussion
Ok


👀 @hatboysam-test it's your turn please take a look

Copy link
Contributor

@hatboysam hatboysam left a comment

Choose a reason for hiding this comment

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

Automated comment from CodeApprove (go to review)

✔️ Approved (2 unresolved comments)


In: Discussion
Draft


👀 @hatboysam-test it's your turn please take a look

Copy link
Contributor

@hatboysam hatboysam left a comment

Choose a reason for hiding this comment

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

Automated comment from CodeApprove (go to review)

In: Discussion
Resolved


In: Discussion
Resolved


👀 @hatboysam-test it's your turn please take a look

Copy link
Contributor

@hatboysam hatboysam left a comment

Choose a reason for hiding this comment

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

Automated comment from CodeApprove (go to review)

✔️ Approved (1 unresolved comments)


In: Discussion
OK


👀 @hatboysam-test it's your turn please take a look

Copy link
Contributor

@hatboysam hatboysam left a comment

Choose a reason for hiding this comment

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

Automated comment from CodeApprove (go to review)

✔️ Approved (2 unresolved comments)


In: Discussion
Ok


👀 @hatboysam-test it's your turn please take a look

Copy link
Contributor

@hatboysam hatboysam left a comment

Choose a reason for hiding this comment

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

Automated comment from CodeApprove (go to review)

❌ Undo Approval (2 unresolved comments)

Undo approval


👀 @hatboysam-test it's your turn please take a look

Copy link
Contributor

@hatboysam hatboysam left a comment

Choose a reason for hiding this comment

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

Automated comment from CodeApprove (go to review)

✔️ Approved (3 unresolved comments)


👀 @hatboysam-test it's your turn please take a look

Copy link
Contributor

@hatboysam hatboysam left a comment

Choose a reason for hiding this comment

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

Automated comment from CodeApprove (go to review)

❌ Undo Approval (3 unresolved comments)

Undo approval


👀 @hatboysam-test it's your turn please take a look

Copy link
Contributor

@hatboysam hatboysam left a comment

Choose a reason for hiding this comment

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

Automated comment from CodeApprove (go to review)

✔️ Approved (4 unresolved comments)


👀 @hatboysam-test it's your turn please take a look

Copy link
Contributor

@hatboysam hatboysam left a comment

Choose a reason for hiding this comment

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

Automated comment from CodeApprove (go to review)

❌ Undo Approval (5 unresolved comments)


👀 @hatboysam-test it's your turn please take a look

Copy link
Contributor

@hatboysam hatboysam left a comment

Choose a reason for hiding this comment

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

Automated comment from CodeApprove (go to review)

✔️ Approved (5 unresolved comments)

Undo approval


👀 @hatboysam-test it's your turn please take a look

Copy link
Contributor

@hatboysam hatboysam left a comment

Choose a reason for hiding this comment

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

Automated comment from CodeApprove (go to review)

❌ Undo Approval (5 unresolved comments)

This is a comment with undo approval


👀 @hatboysam-test it's your turn please take a look

Copy link
Contributor

@hatboysam hatboysam left a comment

Choose a reason for hiding this comment

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

Automated comment from CodeApprove (go to review)

✔️ Approved (5 unresolved comments)

Approve with a comment


👀 @hatboysam-test it's your turn please take a look

Copy link
Contributor

@hatboysam hatboysam left a comment

Choose a reason for hiding this comment

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

Automated comment from CodeApprove (go to review)

❌ Undo Approval (5 unresolved comments)

Undo with a comment


👀 @hatboysam-test it's your turn please take a look

Copy link
Contributor

@hatboysam hatboysam left a comment

Choose a reason for hiding this comment

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

Automated comment from CodeApprove (go to review)

Just a comment


👀 @hatboysam-test it's your turn please take a look

Copy link
Contributor

@hatboysam hatboysam left a comment

Choose a reason for hiding this comment

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

Automated comment from CodeApprove (go to review)

✔️ Approved (5 unresolved comments)

Approve with a comment


👀 @hatboysam-test it's your turn please take a look

Copy link
Contributor

@hatboysam hatboysam left a comment

Choose a reason for hiding this comment

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

Automated comment from CodeApprove (go to review)

✔️ Approved (5 unresolved comments)

Approve this


👀 @hatboysam-test it's your turn please take a look

Copy link
Contributor

@hatboysam hatboysam left a comment

Choose a reason for hiding this comment

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

Automated comment from CodeApprove (go to review)

✔️ Approved (2 unresolved comments)


In: Discussion
Resolve


In: Discussion
Resolve


👀 @hatboysam-test it's your turn please take a look

Copy link
Contributor

@hatboysam hatboysam left a comment

Choose a reason for hiding this comment

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

Automated comment from CodeApprove (go to review)

✔️ Approved (1 unresolved comments)

Resolving discussions


👀 @hatboysam-test it's your turn please take a look

Copy link
Contributor

@hatboysam hatboysam left a comment

Choose a reason for hiding this comment

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

Automated comment from CodeApprove (go to review)

In: Discussion
Resolve


👀 @hatboysam-test it's your turn please take a look

Copy link
Contributor

@hatboysam hatboysam left a comment

Choose a reason for hiding this comment

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

Automated comment from CodeApprove (go to review)

Unresolved!


👀 @hatboysam-test it's your turn please take a look

@hatboysam hatboysam dismissed their stale review May 28, 2022 18:32

Reviewer removed on CodeApprove

Copy link
Contributor

@hatboysam hatboysam left a comment

Choose a reason for hiding this comment

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

Automated comment from CodeApprove (go to review)

✔️ Approved (1 unresolved comments)

New approval


👀 @hatboysam-test it's your turn please take a look

Copy link
Contributor

@hatboysam hatboysam left a comment

Choose a reason for hiding this comment

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

Automated comment from CodeApprove (go to review)

Keeping approval


In: Discussion
Resolved


👀 @hatboysam-test it's your turn please take a look

Copy link
Contributor

@hatboysam hatboysam left a comment

Choose a reason for hiding this comment

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

Seeing if comment after approval rmoves it

Copy link
Contributor

@hatboysam hatboysam left a comment

Choose a reason for hiding this comment

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

Automated comment from CodeApprove (go to review)

✔️ Approved (1 unresolved comments)

Justnone comment


In: index.js:

> Line 6
}

Refactor


👀 @hatboysam-test it's your turn please take a look

Copy link
Contributor

@hatboysam hatboysam left a comment

Choose a reason for hiding this comment

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

Automated comment from CodeApprove (go to review)

In: index.js:

> Line 6
}

Resolved


👀 @hatboysam-test it's your turn please take a look

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

Successfully merging this pull request may close these issues.

2 participants