-
-
Notifications
You must be signed in to change notification settings - Fork 12.8k
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
blueprint-compiler: 0.6.0 -> 0.8.1 #238303
Conversation
So this update will break 2 How do i proceed with this? Do i have to patch these packages in this pr? Notify maintainer:
|
Hi @BenediktBroich it seems like "giara" expects version 0.6 of blueprint-compiler, I even tried bumping the version of giara to 1.1.0 but it still relies on version 0.6 of blueprint-compiler. EDIT: it seems like the change is non-trivial and I got giara to work with blueprint 0.8. So I opened PR #238409 which works with both version 0.6 and 0.8. So after that is merged there should no longer be a blocker for your PR. Thanks. |
or mark it broken |
inside this pr? |
a7254c4
to
09ddba2
Compare
09ddba2
to
3e2314b
Compare
fcbc504
to
af6d124
Compare
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.
This should be two commits as far as I can tell.
One marking textpieces broken in the newer version, and one updating blueprint-compiler.
Otherwise this looks good.
af6d124
to
4ba8610
Compare
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.
LGTM
Description of changes
v0.8.1
Things done
sandbox = true
set innix.conf
? (See Nix manual)nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD"
. Note: all changes have to be committed, also see nixpkgs-review usage./result/bin/
)