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

forgejo-runner: 3.5.1 -> 4.0.1 #352928

Merged
merged 1 commit into from
Nov 4, 2024
Merged

Conversation

emilylange
Copy link
Member

@emilylange emilylange commented Nov 1, 2024

https://code.forgejo.org/forgejo/runner/src/tag/v4.0.1/RELEASE-NOTES.md

diff: https://code.forgejo.org/forgejo/runner/compare/v3.5.1...v4.0.1

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.

@alexandru0-dev alexandru0-dev added the 2.status: wait for branch‐off Waiting for the next Nixpkgs branch‐off label Nov 1, 2024
@alexandru0-dev
Copy link
Contributor

4.0.1 includes breaking changes.
Do not merge this PR when breaking changes are not allowed, see #339153.

(Breaking changes are not allowed until November 8, sry if you're going to wait for this PR to be merged)

@emilylange
Copy link
Member Author

4.0.1 includes breaking changes. Do not merge this PR when breaking changes are not allowed, see #339153.

(Breaking changes are not allowed until November 8, sry if you're going to wait for this PR to be merged)

Not sure where you are getting November 8 from, but this is not the correct date.
On November 8 the only branch that will be unrestricted from the breaking change restriction is staging.
This PR here is targeting master, not staging.
master will be unrestricted after branch-off, on November 14.

Additionally, the breaking change you are referring to is technically a security fix.
I just did not feel like flagging it as such via the 1. severity: security label.

And finally, forgejo-runner is a leaf package. No other package depends on it.

I am not willing to go into 24.11 with an EOL version.


I'll look into why the VM test is all of a sudden failing and try to fix it.
I can't remember it failing when I created this PR (otherwise I wouldn't have ticked the "Tested, as applicable" checkbox), but I am able to reproduce it consistently on the machine I am currently on.

When this is sorted, I'll go forward with merging this PR.
If for reasons that aren't clear to me yet this is deemed the wrong call here, I am fine with a revert and will look into the alternatives.

@emilylange emilylange removed the 2.status: wait for branch‐off Waiting for the next Nixpkgs branch‐off label Nov 4, 2024
@emilylange
Copy link
Member Author

Test got fixed in #353670

@ofborg test forgejo-runner

@emilylange
Copy link
Member Author

@ofborg build forgejo-runner.tests

@wegank wegank added the 12.approvals: 1 This PR was reviewed and approved by one reputable person label Nov 4, 2024
@emilylange emilylange merged commit d527044 into NixOS:master Nov 4, 2024
43 of 48 checks passed
@emilylange emilylange deleted the forgejo-runner branch November 4, 2024 18:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
10.rebuild-darwin: 1-10 10.rebuild-darwin: 1 10.rebuild-linux: 1-10 10.rebuild-linux: 1 11.by: package-maintainer This PR was created by the maintainer of the package it changes 12.approvals: 1 This PR was reviewed and approved by one reputable person
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants