Skip to content

nixpkgs-review: 3.0.1 -> 3.1.0#379436

Merged
SuperSandro2000 merged 1 commit intoNixOS:masterfrom
GaetanLepage:nixpkgs-review
Feb 5, 2025
Merged

nixpkgs-review: 3.0.1 -> 3.1.0#379436
SuperSandro2000 merged 1 commit intoNixOS:masterfrom
GaetanLepage:nixpkgs-review

Conversation

@GaetanLepage
Copy link
Contributor

@GaetanLepage GaetanLepage commented Feb 4, 2025

Things done

Changelog: https://github.com/Mic92/nixpkgs-review/releases/tag/3.1.0

  • 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/)
  • 25.05 Release Notes (or backporting 24.11 and 25.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.

@GaetanLepage
Copy link
Contributor Author

nixpkgs-review result

Generated using nixpkgs-review.

Command: nixpkgs-review pr 379436


x86_64-linux

✅ 8 packages built:
  • chirpstack-concentratord
  • gcalcli
  • gcalcli.dist
  • kcl
  • nix-update
  • nix-update.dist
  • nixpkgs-review
  • nixpkgs-review.dist

aarch64-linux

✅ 8 packages built:
  • chirpstack-concentratord
  • gcalcli
  • gcalcli.dist
  • kcl
  • nix-update
  • nix-update.dist
  • nixpkgs-review
  • nixpkgs-review.dist

x86_64-darwin

✅ 7 packages built:
  • gcalcli
  • gcalcli.dist
  • kcl
  • nix-update
  • nix-update.dist
  • nixpkgs-review
  • nixpkgs-review.dist

aarch64-darwin

✅ 7 packages built:
  • gcalcli
  • gcalcli.dist
  • kcl
  • nix-update
  • nix-update.dist
  • nixpkgs-review
  • nixpkgs-review.dist

Copy link
Member

@PerchunPak PerchunPak left a comment

Choose a reason for hiding this comment

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

LGTM

Allows us to merge #377508

@SuperSandro2000 SuperSandro2000 merged commit ff7993f into NixOS:master Feb 5, 2025
46 of 48 checks passed
@GaetanLepage GaetanLepage deleted the nixpkgs-review branch February 5, 2025 10:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants