-
-
Notifications
You must be signed in to change notification settings - Fork 13.7k
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
xonsh: 0.15.1 -> 0.17.0 #314728
xonsh: 0.15.1 -> 0.17.0 #314728
Conversation
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.
Maybe also put the compare link in the PR message:
xonsh/xonsh@0.15.1...1613914
Can this move to |
And using an outdated prompt-toolkit in a wild override stack that will be removed in the next release? Nope. |
0.17.0 was released just now. Let's use this stable version instead. |
Co-authored-by: Anderson Torres <[email protected]> Upstream changes: xonsh/xonsh@0.15.1...0.17.0
LGTM |
Can we backport this to 24.05? |
I'm not so sure about it tho, as it introduces at least one breaking change (xonsh/xonsh#5491) |
Backport failed for Please cherry-pick the changes locally and resolve any conflicts. git fetch origin release-24.05
git worktree add -d .worktree/backport-314728-to-release-24.05 origin/release-24.05
cd .worktree/backport-314728-to-release-24.05
git switch --create backport-314728-to-release-24.05
git cherry-pick -x afc31073a3bb5d61825c51cadc67f75ab4d65ed0 |
Description of changes
Adopted from #305316
Fixes #308868
Fixes #206062
Closes #301670
Upstream changes: xonsh/xonsh@0.15.1...0.17.0
Notably,
~/.xonshrc
is no longer loaded by default in non-interactive mode. See xonsh/xonsh#5491Things done
nix.conf
? (See Nix manual)sandbox = relaxed
sandbox = true
nix-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.