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

Failing test for #152 #286

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Failing test for #152 #286

wants to merge 2 commits into from

Conversation

hakanai
Copy link

@hakanai hakanai commented Mar 13, 2020

I've added a test that I think should pass, but it fails. Now I just have to figure out what is being done wrong.

@nhojpatrick
Copy link
Member

@trejkaz looks good, please can you rebase from master, as hamcrest-core and hamcrest-library have been refactored a lot and also deprecated, so that everything is just in hamcrest.

@hakanai
Copy link
Author

hakanai commented Jul 9, 2020

Rebase seemed suspiciously clean but I guess I'll let Travis be the judge.

@hakanai
Copy link
Author

hakanai commented Jul 9, 2020

It failed, but then when I re-read my ticket description above that is the expected, yes. So the question is now how to get the new test to pass. :)

@nhojpatrick
Copy link
Member

Going to try and kick start hamcrest, so if you want to get it merged, please rebase from the branch v2.3-candidates.
Still trying to understand how has permissions to perform a release.

@nhojpatrick nhojpatrick force-pushed the master branch 2 times, most recently from 9bc653b to e9f7fc8 Compare February 13, 2022 17:06
@hakanai
Copy link
Author

hakanai commented Feb 17, 2022

Probably not a lot of benefit getting this into a release because it still wants the actual fix.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
No open projects
Status: High priority
Development

Successfully merging this pull request may close these issues.

2 participants