Skip to content

Reverts Breaking Changes for a Patch Release #33

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

Merged
merged 3 commits into from
Oct 7, 2017

Conversation

tylergaw
Copy link
Collaborator

@tylergaw tylergaw commented Oct 6, 2017

fixes #32

As noted in that issue breaking changes were made in #19. The idea was to release those in a 2.0, but I missed that and ended up releasing them in 1.3.1.

This PR removes those changes for a quick 1.3.3. This release will include the updates in #26 and #31.

@tylergaw
Copy link
Collaborator Author

tylergaw commented Oct 6, 2017

@ai Does this seem correct?

@ai
Copy link

ai commented Oct 6, 2017

Yeap, great!

@ai
Copy link

ai commented Oct 6, 2017

BTW, also we can in 2.0 you can add color-mod() func name since spec was changed.

@tylergaw
Copy link
Collaborator Author

tylergaw commented Oct 6, 2017

Cool, thanks.

On color-mod I'm still confused on the spec. I mentioned in this issue #29 that the naming is unclear. I wonder if the spec just needs updated still? Wonder if @tabatkins could shed some light on that for us?

@tabatkins
Copy link

Basically ignore the color-mod() function for now. It needs to be substantially revamped.

@ai
Copy link

ai commented Oct 7, 2017

@tabatkins thanks!

@tylergaw
Copy link
Collaborator Author

tylergaw commented Oct 7, 2017

Excellent, thanks @tabatkins!

OK, I'm going to go ahead and merge / release this. Then follow it up with the 2.0 release we discussed.

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.

Breaking change with major release
3 participants