Skip to content
This repository has been archived by the owner on Sep 30, 2020. It is now read-only.

Fix StreamMatchers.equalTo() types #22

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

Fix StreamMatchers.equalTo() types #22

wants to merge 4 commits into from

Conversation

runeflobakk
Copy link
Contributor

@runeflobakk runeflobakk commented Apr 11, 2020

This is a small addition to #19 to fix some typing. Se discussion here #18 (comment)

Fixes #18

@runeflobakk runeflobakk changed the title Fix equalto types Fix StreamMatchers.equalTo() types Apr 13, 2020
This is peculiar. It seems the type inference through multiple levels of
method invokations is better with Java 8 than Java 11.

Anyway, the outermost is(..) invocation is redundant (and in my opinion
also decreases readability), so they are removed to make the code
compile on Java 11.
@CliveEvans
Copy link
Contributor

Looks good but we have no build status. What happened to Travis?

@runeflobakk
Copy link
Contributor Author

runeflobakk commented May 13, 2020

I had to do some changes in #23 to get the Travis build to work for my fork
https://travis-ci.com/github/runeflobakk/java-8-matchers

Build on Travis on OpenJDK 8 (rationale in commit message) 1070dd7

Other changes in .travis.yml

  • Only do one build on Travis a2b6a34
  • Cache .m2 to speed up Travis building 91a644e

@bruceeddy
Copy link
Contributor

bruceeddy commented May 13, 2020 via email

@runeflobakk
Copy link
Contributor Author

Great!
I have no shame, so I am nominating myself 😅

@CliveEvans
Copy link
Contributor

I'm happy to come back on board ;-)

@CliveEvans
Copy link
Contributor

I had to do some changes in #23 to get the Travis build to work for my fork
https://travis-ci.com/github/runeflobakk/java-8-matchers

Ok, let's get that one in first then.

looks meaningfully at @bruceeddy

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

StreamMatchers.empty() and StreamMatchers.equalTo() return Matcher<BaseStream<T, S>>
4 participants