Skip to content

Conversation

@castavridis
Copy link

No description provided.

Copy link
Author

@castavridis castavridis left a comment

Choose a reason for hiding this comment

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

Test review of changes.

Copy link
Author

@castavridis castavridis left a comment

Choose a reason for hiding this comment

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

Walking through reviewing code.

@@ -1,3 +1,4 @@
<!-- NOTE: Testing code review flow.-->
Copy link
Author

Choose a reason for hiding this comment

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

Reasoning for change could go here?

<!-- NOTE: Testing code review flow.-->
<html>
<head>
<title>Git Cheat Sheet</title>
Copy link
Author

Choose a reason for hiding this comment

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

Can comment on lines of code outside of the commit?

Copy link
Author

Choose a reason for hiding this comment

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

But doesn't work on lines below the code fold. Only lines close to the change.

@castavridis
Copy link
Author

Proposed PR flow for code review:

  • Fork reviewee's repo.
  • Comment areas for improvement in source files. (Can do this online on GitHub or locally).
  • Create pull request.
  • Add more context in PR review, if necessary.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant