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

Parse error for compiler version 0.13.x on v3.0.0 tag #29

Closed
joneshf opened this issue Jul 30, 2019 · 6 comments
Closed

Parse error for compiler version 0.13.x on v3.0.0 tag #29

joneshf opened this issue Jul 30, 2019 · 6 comments

Comments

@joneshf
Copy link

joneshf commented Jul 30, 2019

Howdy. I was looking into whether the tag at v3.0.0 could run on 0.13.x. Looks like there's one small thing that doesn't parse. It should be a non-breaking change as it parses with both 0.12.x and 0.13.x (unless I'm forgetting something). Would you be willing to accept a PR for the change and push out a v3.0.1? If so, it looks like a branch has to be made starting at v3.0.0, then I should be able to make a PR. If not, no big.

@natefaubion
Copy link
Collaborator

Yes, that would be great. I'm not sure how this didn't get caught leading up to 0.13.

@joneshf
Copy link
Author

joneshf commented Jul 30, 2019

Great! Lemme know when a branch is made so I can submit a PR.

v3.0.0 is over a year old. The line was removed some time ago. Newer versions are compatible with compiler version 0.13.x. We ran into this because we're still on purescript-halogen v4.0.0 and it depends on v3.0.0 of this package.

Specifically, it depends on ElemSpec, so newer versions of this package would require changes to purescript-halogen based on the v4.0.0 tag. That seems more intrusive, though it would mean the updates from v4.0.0 on of this package would make its way into v4.x.x series of purescript-halogen.

@garyb
Copy link
Member

garyb commented Aug 2, 2019

Sorry, I only just realised you'd asked for a branch to be made for this. It's there now! https://github.com/slamdata/purescript-halogen-vdom/tree/v3-backport

@joneshf
Copy link
Author

joneshf commented Aug 8, 2019

No worries. Thanks for the branch!

@garyb
Copy link
Member

garyb commented Aug 8, 2019

Released as v3.0.1. I've left the branch there, so if this doesn't do the trick we can always make further releases based on it 😉.

@garyb garyb closed this as completed Aug 8, 2019
@joneshf
Copy link
Author

joneshf commented Aug 8, 2019

Thanks!

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

No branches or pull requests

3 participants