Skip to content

Conversation

@hatboysam
Copy link
Contributor

@hatboysam hatboysam commented Jan 3, 2021

Just a quick test.

https://www.google.com

@hatboysam-test mentioning you

Copy link

@codeapprove-dev codeapprove-dev bot left a comment

Choose a reason for hiding this comment

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

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

@codeapprove-dev
Copy link

@hatboysam: ✔️ Approved

@codeapprove-dev
Copy link

@hatboysam: ⏳ Needs Approval

Copy link

@codeapprove-dev codeapprove-dev bot left a comment

Choose a reason for hiding this comment

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

Status

⏳ Needs Review

Reviewers

None

Comments

File # Threads # Unresolved
README.md 2 2

@codeapprove-dev
Copy link

@hatboysam: ✔️ Approved

@codeapprove-dev
Copy link

hatboysam added comments: 2 new, 0 replies

Copy link

@codeapprove codeapprove bot left a comment

Choose a reason for hiding this comment

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

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

@codeapprove
Copy link

codeapprove bot commented Jan 9, 2021

hatboysam added comments: 1 new, 0 replies

Copy link

@codeapprove codeapprove bot left a comment

Choose a reason for hiding this comment

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

Status

⏳ Needs Review

Reviewers

None

Comments

File # Threads # Unresolved
(none) 1 0

@codeapprove
Copy link

codeapprove bot commented Jan 9, 2021

hatboysam added comments: 0 new, 1 replies

@codeapprove-dev
Copy link

@hatboysam it's your turn, please take a look


🔗 View this review on CodeApprove

@codeapprove-dev
Copy link

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


🔗 View this review on CodeApprove

@codeapprove-dev
Copy link

New Review: ✔️ hatboysam-test approved this PR
Review Status: ❌ Needs Resolution (1 unresolved comments)
Assignees: @hatboysam it's your turn please take a look

hatboysam-test added new comments:


File: README.md

L5: Test another new PR.

Did you mean to add this line?


🔗 View this review on CodeApprove

@codeapprove-dev
Copy link

Review Status: ✔️ Approved
Assignees: @hatboysam it's your turn please take a look

hatboysam added new comments:


File: README.md

L5: Test another new PR.

Yes but thanks for checking


🔗 View this review on CodeApprove

Copy link
Collaborator

@alicethecoder alicethecoder left a comment

Choose a reason for hiding this comment

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

Review Status: ✔️ Approved
Assignees: @hatboysam it's your turn please take a look

New comments:


Discussion

Quick new review.


🔗 View this review on CodeApprove

Copy link
Collaborator

@alicethecoder alicethecoder left a comment

Choose a reason for hiding this comment

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

My Review: ✔️ Approved
Review Status: ✔️ Approved
Assignees: @hatboysam it's your turn please take a look

New comments:


Discussion

Another


🤖 This is an automated comment
🔗 View this review on CodeApprove

Copy link
Collaborator

@alicethecoder alicethecoder left a comment

Choose a reason for hiding this comment

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

My Review: ⏳ Request Changes
Review Status: ⏳ Needs Approval
Assignees: @hatboysam-test it's your turn please take a look


🤖 This is an automated comment
🔗 View this review on CodeApprove

@codeapprove-dev
Copy link

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


🤖 This is an automated comment
🔗 View this review on CodeApprove

Copy link
Collaborator

@alicethecoder alicethecoder left a comment

Choose a reason for hiding this comment

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

✔️ Approved (1 unresolved comments)
@hatboysam it's your turn please take a look

New comments:


File: README.md

L5: Test another new PR.

Approve with an unresolved


🤖 This is an automated comment
🔗 View this review on CodeApprove

Copy link
Collaborator

@alicethecoder alicethecoder left a comment

Choose a reason for hiding this comment

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

@hatboysam it's your turn please take a look

New comments:


File: README.md

L5: Test another new PR.

Finish it


🤖 This is an automated comment
🔗 View this review on CodeApprove

Copy link
Collaborator

@alicethecoder alicethecoder left a comment

Choose a reason for hiding this comment

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


🤖 This is an automated comment
🔗 View this review on CodeApprove

@codeapprove-dev
Copy link

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


🤖 This is an automated comment
🔗 View this review on CodeApprove

Copy link
Collaborator

@alicethecoder alicethecoder left a comment

Choose a reason for hiding this comment

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

✔️ Approved
❌ Undo Approval
@hatboysam it's your turn please take a look


🤖 This is an automated comment
🔗 View this review on CodeApprove

Copy link
Contributor Author

@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: README.md:

> Line 3
CodeApprove meta repo for good testing.

Another


In: README.md:

> Line 3
CodeApprove meta repo for good testing.

Quick reply


In: README.md:

> Line 3
CodeApprove meta repo for good testing.

Another

Copy link
Contributor Author

@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: README.md:

> Line 5
Test another new PR.

One


In: README.md:

> Line 5
Test another new PR.

Two


In: README.md:

> Line 5
Test another new PR.

Three


In: README.md:

> Line 5
Test another new PR.

Four


In: README.md:

> Line 5
Test another new PR.

Five

Copy link
Contributor Author

@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: README.md:

> Line 1
# meta

Unresolved


In: README.md:

> Line 1
# meta

Reply


In: README.md:

> Line 1
# meta

Reply


In: README.md:

> Line 1
# meta

Fourth

Copy link
Contributor Author

@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: README.md:
Single comment

# meta

CodeApprove meta repo for testing.
CodeApprove meta repo for good testing.
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Leaving a comment.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

With a reply on the same thread.

@@ -1,3 +1,5 @@
# meta

Copy link
Contributor Author

Choose a reason for hiding this comment

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

And another comment.

@hatboysam hatboysam assigned hatboysam and alicethecoder and unassigned hatboysam Jul 3, 2021
@codeapprove-dev
Copy link

Automated comment from CodeApprove (go to review)

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

1 similar comment
Copy link
Contributor Author

Automated comment from CodeApprove (go to review)

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

Copy link
Contributor Author

Automated comment from CodeApprove (go to review)

📥 @hatboysam-test you've been cc'ed on this review, check it out

Copy link
Contributor Author

Automated comment from CodeApprove (go to review)

⏳ @hatboysam-test please review this Pull Request

@codeapprove-dev
Copy link

Automated comment from CodeApprove (go to review)

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

Copy link
Contributor Author

Automated comment from CodeApprove (go to review)

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

Copy link
Collaborator

@alicethecoder alicethecoder 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
Whole new thread.


In: README.md:

> Line 1
# meta

Inline response.


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

Copy link
Contributor Author

Automated comment from CodeApprove (go to review)

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

Copy link
Contributor Author

@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: README.md:

> Line 3
CodeApprove meta repo for good testing.

Needs attention.

Copy link
Contributor Author

@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: README.md:
Resolved

Copy link
Contributor Author

@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: README.md:
Done

Copy link
Contributor Author

Automated comment from CodeApprove (go to review)

⏳ @hatboysam-test please review this Pull Request

Copy link
Contributor Author

Automated comment from CodeApprove (go to review)

⏳ @hatboysam-test please review this Pull Request

Copy link
Collaborator

@alicethecoder alicethecoder 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)


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

Copy link
Collaborator

@alicethecoder alicethecoder 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 (1 unresolved comments)

Copy link
Collaborator

@alicethecoder alicethecoder 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 (1 unresolved comments)

Copy link
Collaborator

@alicethecoder alicethecoder 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: README.md:

> Line 7
Add another commit.

Comment


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

Copy link
Collaborator

@alicethecoder alicethecoder 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)


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

Copy link
Contributor Author

@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.

Commented

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.

3 participants