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

Bumped Elm version requirement to 0.16+ #395

Closed
wants to merge 1 commit into
base: master
from

Conversation

Projects
None yet
2 participants
@hughfdjackson

hughfdjackson commented Sep 6, 2015

Recent changes merged into master don't compile with 0.15.x. To
avoid confusion for contributers, this commit requires Elm 0.16+.

The specific PRs that seem to have caused problems for me are #382, which results in a cyclical dependency error, and #375, which complains at duplicate keys in Records for Graphics.Element's width, etc.

Bumped Elm version requirement to 0.16+
Recent changes merged into master don't compile with 0.15.x. To
avoid confusion for contributers, this commit requires Elm 0.16+.
@evancz

This comment has been minimized.

Show comment
Hide comment
@evancz

evancz Sep 7, 2015

Member

Right now nothing will compile with the master branch of elm-compiler. Making this change means we need to do a version bump that will ripple through all the elm repos. elm-package depends on 0.16, etc.

Due to some silly things with cabal, I tend to wait til the last moment to bump all the dependencies.

So given that nothing can be built with the latest compiler anyway, can we wait on this a bit?

Member

evancz commented Sep 7, 2015

Right now nothing will compile with the master branch of elm-compiler. Making this change means we need to do a version bump that will ripple through all the elm repos. elm-package depends on 0.16, etc.

Due to some silly things with cabal, I tend to wait til the last moment to bump all the dependencies.

So given that nothing can be built with the latest compiler anyway, can we wait on this a bit?

@hughfdjackson

This comment has been minimized.

Show comment
Hide comment
@hughfdjackson

hughfdjackson Sep 8, 2015

@evancz :) Absolutely. Closing this PR.

hughfdjackson commented Sep 8, 2015

@evancz :) Absolutely. Closing this PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment