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

Test detecting base-only changes when rebasing #79

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

Conversation

pkaminski
Copy link
Owner

@pkaminski pkaminski commented Sep 29, 2022

This change is Reviewable

@pkaminski pkaminski force-pushed the rebasing branch 2 times, most recently from 8762222 to 0580bca Compare September 29, 2022 05:03
@pkaminski
Copy link
Owner Author

Random comment from GitHub.

Copy link
Owner Author

@pkaminski pkaminski left a comment

Choose a reason for hiding this comment

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

Reviewed 2 of 2 files at r1.
Reviewable status: 0 of 2 files reviewed, all discussions resolved (waiting on @pkaminski)

Copy link
Owner Author

@pkaminski pkaminski left a comment

Choose a reason for hiding this comment

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

qqasdf

Reviewed 1 of 2 files at r2.
Reviewable status: 1 of 2 files reviewed, all discussions resolved (waiting on @pkaminski)

Copy link
Owner Author

@pkaminski pkaminski left a comment

Choose a reason for hiding this comment

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

Reviewable status: 1 of 2 files reviewed, all discussions resolved (waiting on @pkaminski)


-- commits line 2 at r2:
new message

Copy link
Owner Author

@pkaminski pkaminski left a comment

Choose a reason for hiding this comment

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

Reviewable status: 1 of 2 files reviewed, all discussions resolved (waiting on @pkaminski)


-- commits line 1 at r2:
qwe

Copy link
Owner Author

@pkaminski pkaminski left a comment

Choose a reason for hiding this comment

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

Reviewable status: 1 of 2 files reviewed, all discussions resolved (waiting on @pkaminski)


-- commits line 2 at r2:
we

Copy link
Owner Author

@pkaminski pkaminski left a comment

Choose a reason for hiding this comment

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

Reviewable status: 1 of 2 files reviewed, all discussions resolved (waiting on @pkaminski)


-- commits line 1 at r2:
wer

Copy link
Owner Author

@pkaminski pkaminski left a comment

Choose a reason for hiding this comment

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

Reviewable status: 1 of 2 files reviewed, all discussions resolved (waiting on @pkaminski)


-- commits line 1 at r2:
qwer

Copy link
Owner Author

@pkaminski pkaminski left a comment

Choose a reason for hiding this comment

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

Reviewable status: 1 of 2 files reviewed, all discussions resolved (waiting on @pkaminski)


-- commits line 1 at r2:
asdf

Copy link
Owner Author

@pkaminski pkaminski left a comment

Choose a reason for hiding this comment

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

Reviewable status: 1 of 2 files reviewed, all discussions resolved (waiting on @pkaminski)


-- commits line 1 at r2:
qewr

Copy link
Owner Author

@pkaminski pkaminski left a comment

Choose a reason for hiding this comment

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

Reviewable status: 1 of 2 files reviewed, 3 unresolved discussions


-- commits line 0 at r1:
asdf


-- commits line 1 at r2:
nit:

sfdgadf

a
sdf

asf

foo 6


-- commits line 2 at r2:
Nit:

Copy link
Owner Author

@pkaminski pkaminski left a comment

Choose a reason for hiding this comment

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

Reviewed all commit messages.
Reviewable status: 1 of 2 files reviewed, all discussions resolved (waiting on @pkaminski)

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