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 value-level kind-projector syntax #3438

Merged
merged 1 commit into from
May 28, 2020
Merged

Conversation

travisbrown
Copy link
Contributor

This is a trivial syntax switch away from kind-projector lambdas that aren't supported in Dotty.

This got merged in #3374—I just started refreshing #3269 and noticed it again. It's the only one that's snuck in, and we might as well go ahead and change it.

@travisbrown travisbrown assigned fthomas and LukaJCB and unassigned fthomas and LukaJCB May 28, 2020
@travisbrown
Copy link
Contributor Author

Sorry, @LukaJCB, @fthomas—meant to request reviews, not assign you.

@codecov-commenter
Copy link

codecov-commenter commented May 28, 2020

Codecov Report

Merging #3438 into master will increase coverage by 0.00%.
The diff coverage is 100.00%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #3438   +/-   ##
=======================================
  Coverage   91.64%   91.64%           
=======================================
  Files         381      381           
  Lines        8268     8269    +1     
  Branches      225      225           
=======================================
+ Hits         7577     7578    +1     
  Misses        691      691           
Flag Coverage Δ
#scala_version_212 91.63% <100.00%> (+<0.01%) ⬆️
#scala_version_213 91.43% <100.00%> (+<0.01%) ⬆️
Impacted Files Coverage Δ
core/src/main/scala/cats/data/EitherT.scala 98.51% <100.00%> (+<0.01%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 9c20c7b...620961e. Read the comment docs.

@travisbrown travisbrown merged commit a0f3534 into master May 28, 2020
@LukaJCB LukaJCB deleted the topic/remove-lambda branch May 28, 2020 20:46
@travisbrown travisbrown added this to the 2.2.0-M3 milestone May 31, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants