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.

Sign up for GitHub

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

Jump to bottom

remove bijection #2

Closed
johnynek opened this issue Jan 29, 2015 · 1 comment · Fixed by #51
Closed

remove bijection #2

johnynek opened this issue Jan 29, 2015 · 1 comment · Fixed by #51
Assignees
@non

Comments

@johnynek
Copy link
Contributor

or really any typeclass. We got burned with an early version of twitter/bijection that did this. It meant having an implicit bijection in scope gave you an implicit conversion.
@non
Copy link
Contributor

non commented Jan 29, 2015

That is a great point.

Also I think we may remove this type, since it's not that important in this case, and other folks might want more fleshed-out libraries (e.g. twitter-bijection).

@stew stew added the ready label Feb 2, 2015
@stew stew changed the title might not want bijection to extend function remove bijection Feb 2, 2015
non added a commit that referenced this issue Feb 3, 2015
@non
Remove Bijection (i.e. A <=> B) and update signatures.
3b2a2bc 
Fixes #2.
@non non mentioned this issue Feb 3, 2015
Merged
@non non self-assigned this Feb 3, 2015
@non non added in progress and removed ready labels Feb 3, 2015
@stew stew closed this as completed in #51 Feb 3, 2015
@stew stew removed the in progress label Feb 3, 2015
stew added a commit that referenced this issue Feb 11, 2015
@stew
Merge pull request #2 from tpolecat/new-website-craps
66c5a9e 
New website craps
adelbertc added a commit that referenced this issue Aug 14, 2015
@adelbertc
Merge pull request #2 from InTheNow/xor-recover
c45866d 
Fix sbt scala version for Travis-CI
@adelbertc adelbertc mentioned this issue May 31, 2016
Closed
ceedubs pushed a commit that referenced this issue Jun 27, 2016
@non
Merge pull request #2 from ceedubs/doc-overhaul-design
39eb009 
Move build instructions to contributing guide
@adelbertc adelbertc mentioned this issue Aug 19, 2016
Merged
kailuowang pushed a commit to kailuowang/cats that referenced this issue Oct 31, 2017
Merge pull request typelevel#2 from kailuowang/LukaJCB-add-sorted-ins…
9f0ef36 
…tances

move Map and Set instances into alleycats
kailuowang pushed a commit to kailuowang/cats that referenced this issue Dec 5, 2017
@SystemFw
Merge pull request typelevel#2 from kailuowang/SystemFw-feature/rethrow
652ed65 
fix build
@rossabaker rossabaker mentioned this issue Dec 17, 2020
Draft
johnynek pushed a commit that referenced this issue May 1, 2024
@TimWSpence
Merge pull request #2 from valencik/more-chain
6cb787d 
Use `Chain` in `traverseDirectly` helpers
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@non non

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Remove Bijection (i.e. A <=> B) and update signatures. typelevel/cats
3 participants
@stew @johnynek @non