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

Add bifold to Bifoldable close #2947. #2962

Closed

Conversation

takayahilton
Copy link
Contributor

No description provided.

@kailuowang kailuowang added this to the 2.1-RC1 milestone Aug 2, 2019
@kailuowang
Copy link
Contributor

kailuowang commented Aug 2, 2019

We can't break BC yet, but we should be able to allow this soon, so I scheduled it to 2.1. Also it would be nice to have some tests.

@codecov-io
Copy link

codecov-io commented Aug 5, 2019

Codecov Report

Merging #2962 into master will decrease coverage by 0.06%.
The diff coverage is 0%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #2962      +/-   ##
==========================================
- Coverage   93.99%   93.92%   -0.07%     
==========================================
  Files         369      371       +2     
  Lines        7056     7062       +6     
  Branches      179      193      +14     
==========================================
+ Hits         6632     6633       +1     
- Misses        424      429       +5
Impacted Files Coverage Δ
core/src/main/scala/cats/Bifoldable.scala 83.33% <0%> (-7.58%) ⬇️
core/src/main/scala/cats/syntax/option.scala 100% <0%> (ø) ⬆️
testkit/src/main/scala/cats/tests/CatsSuite.scala 33.33% <0%> (ø)
...stkit/src/main/scala/cats/tests/CatsEquality.scala 66.66% <0%> (ø)

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 a786f15...b83fa3b. Read the comment docs.

@takayahilton takayahilton force-pushed the Add-bifold-to-Bifoldable branch from 01b2ba4 to b83fa3b Compare August 5, 2019 13:35
@travisbrown
Copy link
Contributor

I just noticed this PR, which was also done in #3088, which was just merged. I'm going to close this one, but thanks, @takayahilton, and I'm sorry we didn't see this one.

I'm not really sure the test here is needed, since it's just checking the default implementation, and I wouldn't really expect Bifoldable implementers to override this method.

@travisbrown travisbrown closed this Nov 5, 2019
@travisbrown travisbrown removed this from the 2.1.0-RC1 milestone Nov 6, 2019
@takayahilton takayahilton deleted the Add-bifold-to-Bifoldable branch August 7, 2020 04:23
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.

4 participants