-
Notifications
You must be signed in to change notification settings - Fork 3.2k
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
Prune unmaintained dependencies #11120
Comments
Running list of unmaintained go packages in Argo Workflows dependencies:
|
For logging, in Go 1.21 there will be a structured logger in the standard library. Proposal: https://www.reddit.com/r/golang/comments/11sdqia/slog_proposal_accepted_for_go_121. You can even use it now via golang.org/x/exp/slog. We implemented a logging interface based off of https://github.com/go-logr/logr. And are able to switch to the new standard library logger without changing downstream modules use of our logging library. |
@JPZ13 Just out of curiosity what metric did you use to determine which packages are unmaintained? |
@tidwall - eyeballed a few and then also used this: https://isitmaintained.com/ gjson was showing up as unmaintained, so I included it. Happy to scratch from the list if it's a false positive |
@JPZ13 Thanks for the info. I consider gjson a maintained project. |
You can find EOL packages by using https://github.com/xeol-io/xeol. This can be added to CI with relative ease for ongoing reporting. |
Gorilla projects should be back alive. gorilla/mux#707 (comment) |
That's cool @tico24 - I've built a similar thing at https://dmd.tanna.dev using https://endoflife.date as well as community-sourced data! |
@agilgur5 wdyt about moving to https://github.com/casbin/govaluate for the meanwhile? |
Removing bellows -> #13591 |
I wouldn't think about it too much as we're going to remove it soon, c.f. #9529, #7831 |
Summary
Similar to #11053 that @isubasinghe reported, we have many dependencies in our go.mod file (and I assume more in the package.json) that are no longer being actively maintained. We should start culling these dependencies. It gets us the following advantages:
Message from the maintainers:
Love this enhancement proposal? Give it a 👍. We prioritise the proposals with the most 👍.
The text was updated successfully, but these errors were encountered: