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.

Sign up for GitHub

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

Jump to bottom

cleanup definition of dependencies #529

Merged
merged 3 commits into from
Mar 10, 2023
Merged

cleanup definition of dependencies #529

merged 3 commits into from
Mar 10, 2023

Conversation

linki
Copy link
Owner

@linki linki commented Feb 8, 2023

Just cleaning up go.mod

@linki
Copy link
Owner Author

linki commented Feb 8, 2023

Version 2.3.x of kingpin changed how Default("-1s") is treated. We need to change it if we want to adopt this version. See alecthomas/kingpin#333

@SuperQ
Copy link

SuperQ commented Mar 7, 2023

@linki Try go get github.com/alecthomas/kingpin/v2@master.

@linki linki merged commit da28115 into master Mar 10, 2023
@linki linki deleted the cleanup-deps branch March 10, 2023 10:34
@linki
Copy link
Owner Author

linki commented Mar 10, 2023

@SuperQ Thanks for pointing that out! 🙇

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@linki @SuperQ