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 @Deprecated annotations on boolean instances #949

Merged
merged 1 commit into from
Jul 22, 2018

Conversation

BranislavLazic
Copy link
Contributor

@BranislavLazic BranislavLazic commented Jul 22, 2018

  • Add @deprecated annotation on arrow-core BooleanInstances
  • Add @deprecated annotation on arrow-instances-core boolean show and eq
    functions

Closes #943

* Add @deprecated annotation on arrow-core BooleanInstances
* Add @deprecated annotation on arrow-instances-core boolean show and eq
functions
@codecov
Copy link

codecov bot commented Jul 22, 2018

Codecov Report

Merging #949 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##             master     #949   +/-   ##
=========================================
  Coverage     46.21%   46.21%           
  Complexity      642      642           
=========================================
  Files           304      304           
  Lines          7859     7859           
  Branches        806      806           
=========================================
  Hits           3632     3632           
  Misses         3906     3906           
  Partials        321      321
Impacted Files Coverage Δ Complexity Δ
...es-core/src/main/kotlin/arrow/instances/boolean.kt 0% <ø> (ø) 0 <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 bfb9d78...af9434e. Read the comment docs.

@pakoito pakoito merged commit 947484e into arrow-kt:master Jul 22, 2018
@pakoito
Copy link
Member

pakoito commented Jul 22, 2018

Awesome, thanks for adding it :D We'll be ready for 1.3 now!

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.

3 participants