-
Notifications
You must be signed in to change notification settings - Fork 17
fixed deprecation warnings for julia 0.7 #26
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
Conversation
|
It seems you need to generate a Project.toml file. More info here : |
|
In your commit, at .travis.yml, Julia v0.6 is specified, and travis-ci log details show that julia v0.6.4 is used for Travis-CI build. That seems to generate errors in Travis-CI build. |
|
thanks for the pointers, i updated the .travis.yml and the CI test has passed, |
|
It will be nice if @bicycle1885 could merge this and publish a tag for this version because there is currently no published CLI argument parser library for Julia 0.7/1.0 (ArgParse.jl also need Julia 0.7/1.0 testing) |
|
Any news @bicycle1885 ? |
|
@yadongli just a small Git/Github tip... when submitting a PR you should do it from an other branch than |
|
If @bicycle1885 is too busy to maintain this package, I would be happy to help out. I know Kenta has not been very responsive on GitHub recently, so if we can spread out the maintenance burden a bit that might help everybody. |
|
@staticfloat That will be very nice if you could tackle this but one of members of docopt Github org should grant you collaborator access to this repository to speed up things. |
|
Hi, I'm very sorry for not being responsive these days, and I think it will continue for a while. Fortunately, @staticfloat has volunteered to maintain this package and so I've added him as an admin. I'm going to get back soon or later but I think co-maintaining the package is a good idea. Thank you very much, @staticfloat! |
|
Thanks Kenta, I know that life outside of Julia-land can become very busy at times. :) I'm going to merge this and tag a new release. Thanks @yadongli for doing the work of upgrading the syntax. |
all tests has passed, please review and merge.