-
-
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
stylua: 0.14.3 -> 0.15.0 #192279
stylua: 0.14.3 -> 0.15.0 #192279
Conversation
7fe89ba
to
46eff14
Compare
Upstream's binaries are built with |
I will turn on lua54 and luau then, not turning on lua52 and lua53 by default so it is easier to do e.g. lua51 only |
46eff14
to
2eb8b83
Compare
I'd rather we follow upstream binaries and do |
the lua54 feature implies lua53, and the same for lua52 |
How about something like this so we don't need to pick and/or track specific features and people can still override it if they need to? diff --git a/pkgs/development/tools/stylua/default.nix b/pkgs/development/tools/stylua/default.nix
index 492263f2c36..ee997a6839c 100644
--- a/pkgs/development/tools/stylua/default.nix
+++ b/pkgs/development/tools/stylua/default.nix
@@ -1,10 +1,7 @@
{ lib
, rustPlatform
, fetchFromGitHub
-, lua52Support ? false
-, lua53Support ? false
-, lua54Support ? true
-, luauSupport ? true
+, buildFeatures ? "all-features"
}:
rustPlatform.buildRustPackage rec {
@@ -25,10 +22,7 @@ rustPlatform.buildRustPackage rec {
rm .cargo/config.toml
'';
- buildFeatures = lib.optional lua52Support "lua52"
- ++ lib.optional lua53Support "lua53"
- ++ lib.optional lua54Support "lua54"
- ++ lib.optional luauSupport "luau";
+ inherit buildFeatures;
meta = with lib; {
description = "An opinionated Lua code formatter";
|
2eb8b83
to
6e901f5
Compare
I don't think all-features would have worked in this case so I just switched to |
Why not? |
in ci it is taken as a flag ( |
6e901f5
to
2147265
Compare
added a release notes entry since this breaks backwards compatibility |
Set buildflags/checkflags to use |
that sounds like a lot for not a lot of benefit, the newest lua version is 5.4 and there probably won't be much more new features |
Apart from this there a reason not to have 52 and 53 here? If we not going to actually use |
because it is shorter and this is the equivalent, lua54 implies lua53 and lua52 |
Yes, I was asking if that was the only reason. For clarity I'd rather we set all of the features. |
Co-authored-by: zowoq <59103226+zowoq@users.noreply.github.com>
it was the only reason, conditionally setting |
Description of changes
I don't know what the default feature set should be. The original repo has no default features, but I haven't been following the original. I also don't want to break backwards compatibility in nixpkgs, but 5.2 is kinda a weird version to stop at, so I changed to default to 5.4
JohnnyMorganz/StyLua@v0.14.3...v0.15.0
https://github.com/JohnnyMorganz/StyLua/blob/main/CHANGELOG.md
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/
)nixos/doc/manual/md-to-db.sh
to update generated release notes