-
Notifications
You must be signed in to change notification settings - Fork 384
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
(k9s) Update to checksum extension to sha256 #2343
(k9s) Update to checksum extension to sha256 #2343
Conversation
✅ Package verification completed without issues. PR is now pending human review |
Hi @pauby, Is there anything from preventing this to be merged? I am not sure how to find out where the OWNER list is, which you mentioned the last time when the PR wasn't going through, so not sure who is now responsible for approving this. |
@danielkoek The only thing holding up the review and merge is time from somebody to look at it. This is a Community maintained repository so most of it is done in people's spare time. |
The We have introduced a requirement that all packages need to be associated with one or more users that will be considered the primary maintainer of the package. This package currently has no such maintainer. If you are willing to become the primary maintainer of the package, please add yourself to that list in the same format already present in that file. By becoming the primary maintainer, this means you will be responsible for doing reviews of that package (merging the PR will still be us that does). If you are not willing to become the primary maintainer, this PR will most likely be held until a primary maintainer becomes available. Please also make sure you have read through our |
@pauby that makes complete sense! Thank you for letting me know @AdmiringWorm I am more than happy to put myself on the primary maintainer for this package as it is something I work with daily it will be quite easy for me to see if the package is not being updated, I will push a new change to add myself to the list of owners! |
✅ Package verification completed without issues. PR is now pending human review |
Tested the update, install and uninstall and they were fine. Thanks for the work here @danielkoek! |
Description
In pr derailed/k9s#2153 they have changed the extension from txt to sha256, this change will make it possible to update k9s
Types of changes
Checklist: