-
Notifications
You must be signed in to change notification settings - Fork 10
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
Update clap and gtfs structures #196
Conversation
This PR should not include ebb06e8 and other changes to removing legacy stats fields I guess |
oups, I screwd up my git stuff |
e9cda92
to
f24e60d
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do you want to run this in the CI job?
Do you mean cargo audit? It could indeed be nice (but isn't this what is done by dependabot?) |
Yep!
Not sure! Indeed Dependabot may be enough. |
(I can help resolving the conflict if needed.) |
oups no, no problem, I just work on this on my spare time, and completly forgot this PR 😁 |
f24e60d
to
d700eb6
Compare
can we merge this? It should not have any impact on transport.data.gouv.fr if I did not miss something 😜 |
LGTM, maybe 🚢 it early next week? |
d700eb6
to
a3f7022
Compare
a3f7022
to
f7cbcce
Compare
Note: I added the test of |
with with
cargo audit
is fully happy (we were depending on structopt which is now integrated in clap