-
-
Notifications
You must be signed in to change notification settings - Fork 18k
kubevela: init 1.9.8 #274721
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
kubevela: init 1.9.8 #274721
Conversation
|
@bryanasdev000 Review? |
bryanasdev000
left a comment
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.
Overall, LGTM, just a nitpick about ldflags and empty maintainers.
|
@superherointj could you please take a look on the tests? https://github.com/NixOS/nixpkgs/pull/274721/checks?check_run_id=19706550471 |
7b93457 to
f3e68fb
Compare
Fixed. |
bryanasdev000
left a comment
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.
Diff LGTM, but there is a new release out, would be cool to init it on latest.
f3e68fb to
98d5664
Compare
bryanasdev000
left a comment
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.
Diff and ofborg LGTM.
ulrikstrid
left a comment
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.
Builds and some basic commands runs
kubevela: init 1.9.8
Release: https://github.com/kubevela/kubevela/releases/tag/v1.9.8
Description of changes
Things done
nix.conf? (See Nix manual)sandbox = relaxedsandbox = truenix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage./result/bin/)Add a 👍 reaction to pull requests you find important.