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
kube3d: 3.0.2 -> 3.1.0 -> 3.1.1 -> 3.1.2 -> 3.1.3 -> 3.1.4 -> 3.1.5 #99725
Conversation
ab164b3
to
074ad7a
Compare
Friendly bump @kuznero @jlesquembre @ngerstle |
769a846
to
1be5c5e
Compare
I've rebased on master & corrected my mistake with the k3sVersion having the |
This pull request has been mentioned on NixOS Discourse. There might be relevant details there: |
@06kellyjac Thanks for your PR, I didn't have time to test it, but since it's a simple version dump , it should be ok |
Thanks @jlesquembre would you mind if I add myself as a maintainer too? |
@06kellyjac go for it, better to have more maintainers :-) |
Added 🙂 |
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.
I notice that k3d added completions for fish, can you add it too?
for shell in bash zsh fish; do
Other than that, all looks good, tested it locally
Sure, I'll pop that in tomorrow morning |
fish completion was added in 3.0.2 https://github.com/rancher/k3d/releases/tag/v3.0.2
Also bumped k3sVersion
Add jk (myself) as a maintainer to help keeping kube3d working and up-to-date.
e797519
to
aaa2cf6
Compare
/marvin opt-in |
@06kellyjac I was trying to use marvin: https://github.com/timokau/marvin-mk2/blob/master/USAGE.md |
/marvin opt-in |
Hi! I'm an experimental bot. My goal is to guide this PR through its stages, hopefully ending with a merge. You can read up on the usage here. |
/status needs_merger |
The PR author cannot set the status to If you are not the PR author and you are reading this, please review the usage of this bot. You may be able to help. Please make an honest attempt to resolve all outstanding issues before setting to |
/status needs_merger |
Theres also a discourse thread for PRs that are reviewed but need a merger. I'd grab the link but I'm on mobile atm. |
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.
Thanks!
Marvin is intended to be a better replacement for that. It
Its still experimental though and will likely remain that way for quite a while. |
Does marvin have a dashboard too or is it just good at only assigning tags if they're valid & replying well to incorrect commands? Edit: reading through the marvin repo and the RFCs now |
I'm not sure what you mean. There is no dashboard. What information would you want displayed? If you want to find PRs to review, you can use this filter: needs_reviewer It usually does not reply to misformatted commands (it should eventually do that, but doesn't for now). The best source of "end user" documentation is probably the bot's "profile" right now. |
Yeah I was just wondering if it was adding good tags for filtering or if there was something else it was providing for nixpkgs pr observability that i didnt know about I dont mean misformatted. I mean it told me when i couldn't change to awaiting merger and that it wanted the reviewer to post it. Which was helpful |
If i put /m*rvin opt-in in the original PR description will it work or does it have to be a comment? |
Motivation for this change
Bump kube3d/k3d to
3.1.0
3.1.1
3.1.2
3.1.3
3.1.4
3.1.5
Things done
Updated version, k3s version, and src sha256
I matched the k3s version to the one output by the GH release binary, hopefully that's correct.
@kuznero @jlesquembre @ngerstle
sandbox
innix.conf
on non-NixOS linux)nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
./result/bin/
)nix path-info -S
before and after)