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

Support Julia 0.5, enable tests on nightlies #1

Merged
merged 1 commit into from
Mar 14, 2016

Conversation

nalimilan
Copy link
Member

Since you mentioned possible features only available in Julia 0.5, I gave it a try and noticed the package doesn't load. This fixes it (and the tests pass).

There are still warnings about the deprecated call syntax, but AFAICT Compat.jl does not support this yet (which is a bit surprising, but that's not essential).

Call overloading does not currently work with abstract types.
JuliaLang/julia#14919
@andyferris
Copy link
Member

This is really great, thanks Milan! I was confused by this but I think I understand better now I re-read through the Julia issue.

andyferris added a commit that referenced this pull request Mar 14, 2016
Support Julia 0.5, enable tests on nightlies
@andyferris andyferris merged commit c33f0eb into JuliaData:master Mar 14, 2016
@andyferris
Copy link
Member

And regarding "possible features only available in Julia 0.5" these are purely speculative. :)

I'm currently doing warm-up by thinking about generic metaprogramming with @pure and I'll possibly try implement a typed-dictionary to play with interfaces. But since 0.5 isn't imminent, this is all academic at this stage.

@nalimilan nalimilan deleted the 0.5 branch March 14, 2016 10:20
@nalimilan
Copy link
Member Author

I was mostly thinking of @pure, which would allow for a nice indexing syntax. With some support from Compat.jl, you could easily offer it on 0.5 while still supporting 0.4.

@nalimilan
Copy link
Member Author

For reference, you should be able to move to the new call overloading syntax using Compat.jl soon: JuliaLang/Compat.jl#181

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.

2 participants