Skip to content
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

hyprland: 0.43.0 -> 0.44.0, hyprpicker: 0.3.0 -> 0.4.1 #346906

Merged
merged 7 commits into from
Oct 9, 2024

Conversation

fufexan
Copy link
Contributor

@fufexan fufexan commented Oct 6, 2024

Update hyprland and hyprpicker. More details in the commit messages.

Hyprland CHANGELOG
Hyprpicker CHANGELOG

Things done

  • Built on platform(s)
    • x86_64-linux
    • aarch64-linux
    • x86_64-darwin
    • aarch64-darwin
  • For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)
    • sandbox = relaxed
    • sandbox = true
  • Tested, as applicable:
  • Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
  • Tested basic functionality of all binary files (usually in ./result/bin/)
  • 24.11 Release Notes (or backporting 23.11 and 24.05 Release notes)
    • (Package updates) Added a release notes entry if the change is major or breaking
    • (Module updates) Added a release notes entry if the change is significant
    • (Module addition) Added a release notes entry if adding a new NixOS module
  • Fits CONTRIBUTING.md.

Add a 👍 reaction to pull requests you find important.

Copy link
Member

@NotAShelf NotAShelf left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm, love u fuf

@Atemu
Copy link
Member

Atemu commented Oct 8, 2024

Could you separate out the reformat part into a separate commit? It's impossible to tell what's going on otherwise.

@JohnRTitor
Copy link
Contributor

Seperate the mold linker commit as well.

@Atemu
Copy link
Member

Atemu commented Oct 9, 2024

I meant the hyprpicker commit.

It's fine to bundle changes and minor fixups related to an update into the update commit but automated reformats generally make it impossible to tell what other changes have actually taken place, so please separate that out into another commit.

Copy link
Member

@Atemu Atemu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

A few nits, otherwise diff LGTM.

pkgs/by-name/hy/hyprland/package.nix Outdated Show resolved Hide resolved
pkgs/by-name/hy/hyprland/package.nix Outdated Show resolved Hide resolved
Use `inherit (lib) x` instead of `lib.x`
Remove no-longer-needed jq
Use importJSON
Leave open the possibility of adding multiple adapters in the future.
@JohnRTitor JohnRTitor merged commit 9fd89c0 into NixOS:master Oct 9, 2024
27 of 28 checks passed
@Atemu
Copy link
Member

Atemu commented Oct 10, 2024

You forgot to update the info.json.

This would have been caught by a simple Hyprland --version check, so perhaps this would be a good time as any to introduce it.

@Atemu
Copy link
Member

Atemu commented Oct 10, 2024

#347624

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants