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

Topic/kernel #978

Closed
wants to merge 20 commits into from
Closed

Topic/kernel #978

wants to merge 20 commits into from

Conversation

has609
Copy link

@has609 has609 commented Apr 13, 2016

No description provided.

milessabin and others added 20 commits January 8, 2016 13:53
Question: is the apply method generated by simulacrum also @inline marked?
As suggested by @mpilquist

sbt validate works locally.
This gets rid of the simulacrum dependency, as well as the strange and
definitely unneeded scoverage dependencies, resulting in a nice and clean
.pom

Code taken from scodec-build. Thanks @mpilquist
Also remove unnecessary extends Serializable
Enforce binary compatibility with MiMa SBT plugin
The removal of provided artefacts is done inline because it is used in only
one place.
Use simulacrum for typeclass boilerplate
@ceedubs
Copy link
Contributor

ceedubs commented Apr 18, 2016

Thanks, @has609!

It looks like there are some merge conflicts.

@non I saw you mention on Gitter that you had a branch where you were working on this. I hate to see people duplicating effort. Where does yours stand?

@julienrf
Copy link
Contributor

julienrf commented May 1, 2016

This one can be closed too, now?

@adelbertc
Copy link
Contributor

Fixed #1001

@adelbertc adelbertc closed this May 1, 2016
@non non deleted the topic/kernel branch September 2, 2016 05:05
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.

7 participants