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

change signature of all comparator methods #327

Closed
wants to merge 1 commit into from

Conversation

bjrke
Copy link

@bjrke bjrke commented Feb 11, 2021

issue: #328

Copy link

@hakanai hakanai left a comment

Choose a reason for hiding this comment

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

Seems right to me.

@bjrke
Copy link
Author

bjrke commented Feb 23, 2021

Seems right to me.

@hakanai I am not allowed to merge this PR, what will happen now with this PR?

@hakanai
Copy link

hakanai commented Mar 4, 2021

You’d need someone with the privileges to come through and merge it.

@bjrke
Copy link
Author

bjrke commented Mar 7, 2021

@nhojpatrick can you merge this PR?

@bjrke
Copy link
Author

bjrke commented Jun 11, 2021

bump

@bjrke
Copy link
Author

bjrke commented Oct 6, 2021

@npryce @sf105 may I ask for a review?
whats the status of this project? it looks dead

@bjrke
Copy link
Author

bjrke commented Nov 26, 2021

I dropped the dependency to hamcrest in our project and therefore will close this pull request. Feel free to reopen it if you need it. I don't need hamcrest anymore.

@bjrke bjrke closed this Nov 26, 2021
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.

2 participants