| Commit message (Expand) | Author | Age | Files | Lines |
|---|
| * | Improve PR merge check for CI (#347786) | Philip Taron | 2024-10-12 | 2 | -0/+117 |
| |\ |
|
| | * | ci/get-merge-commit.sh: Add documentation•••And distinguish exit codes
| Silvan Mosberger | 2024-10-12 | 2 | -5/+59 |
| | * | ci/get-merge-commit.sh: Exit for non-open PRs•••If PRs aren't open (either merged or closed), GitHub never computes
whether the PR is mergeable, so we'd wait forever, which has been
happening:
https://github.com/NixOS/nixpkgs/actions/runs/11279197077/job/31369348101#step:2:59
| Silvan Mosberger | 2024-10-12 | 1 | -0/+8 |
| | * | workflows/nixpkgs-vet: Make merge check script reusable•••This is useful for other workflows as well. Originally I thought it
couldn't be put in the repo, but it can (just needs another checkout)
| Silvan Mosberger | 2024-10-12 | 1 | -0/+55 |
| * | | ci/request-reviews: Avoid duplicates with different casings•••It's possible to have different casings in OWNERS, so we need to handle
that
| Silvan Mosberger | 2024-10-12 | 1 | -6/+7 |
| * | | OWNERS: reference the Emacs team for Emacs stuff (#347973) | Lin Jian | 2024-10-12 | 1 | -3/+5 |
| |\ \ |
|
| | * | | OWNERS: make the Emacs team own Emacs docs | Lin Jian | 2024-10-12 | 1 | -0/+2 |
| | * | | OWNERS: reference the Emacs team for Emacs stuff | Lin Jian | 2024-10-12 | 1 | -3/+3 |
| * | | | ci/request-reviews: fix headRef to point to the PR head•••Previously, headRef points to the master branch of Nixpkgs, which
basically means no code owner review will be requested.
The problem can be verified using the following command.
$ DRY_MODE=1 ./ci/request-reviews/request-reviews.sh NixOS/nixpkgs 347973 ci/OWNERS
[...]
This PR touches 0 files
Requesting reviews from: {
"reviewers": []
}
[...]
Additionally, the comment about conflicts is removed thanks to the
unambiguous way of specifying ref.
| Lin Jian | 2024-10-12 | 1 | -2/+1 |
| * | | | Fix codeowner workflow for branches starting with "pr" (#347831) | Philip Taron | 2024-10-11 | 1 | -4/+3 |
| |\ \ \
| |/ /
|/| | |
|
| | * | | ci/request-reviews: Fix for branches starting with "pr"•••Turns out if :<something> is passed, a local branch is updated, which
can conflict if the PR branch starts with "pr". I tried to avoid that
with the original code but apparently that didn't work!
https://github.com/NixOS/nixpkgs/actions/runs/11284183639/job/31384967152?pr=347822
Fetching the PR commit history
From https://github.com/linj-fork/nixpkgs
* [new branch] pr/kanata-add-version-check -> fork/pr
error: cannot lock ref 'refs/remotes/fork/pr/kanata-add-version-check': 'refs/remotes/fork/pr' exists; cannot create 'refs/remotes/fork/pr/kanata-add-version-check'
! [new branch] pr/kanata-add-version-check -> fork/pr/kanata-add-version-check (unable to update local ref)
error: some local refs could not be updated; try running
| Silvan Mosberger | 2024-10-11 | 1 | -4/+3 |
| | |/ |
|
| * | | OWNERS: reference the ACME team directly (#347832) | Silvan Mosberger | 2024-10-11 | 1 | -1/+1 |
| |\ \
| |/
|/| |
|
| | * | OWNERS: reference the ACME team directly•••We can finally stop leaving @m1cr0man out thanks to
<https://github.com/NixOS/nixpkgs/pull/347610> 🎉
| Emily | 2024-10-11 | 1 | -1/+1 |
| * | | chrony: move to by-name | Adrian Pistol | 2024-10-10 | 1 | -1/+1 |
| |/ |
|
| * | Mass pings be gone, code owners for all (#347610) | Emily | 2024-10-10 | 2 | -2/+396 |
| |\ |
|
| | * | ci/request-reviews: Make wrong base branch message hashes more transparent•••Also, fix the description of the text
| Silvan Mosberger | 2024-10-10 | 1 | -2/+3 |
| | * | CODEOWNERS: Switch to alternate mechanism•••This effectively disables the native GitHub codeowners feature
and enables the new alternate codeowners mechanism introduced in
https://github.com/NixOS/nixpkgs/pull/336261
This means that:
- We can now declare users without write access as code owners!
- Targeting the wrong branch won't trigger mass pings anymore!
| Silvan Mosberger | 2024-10-10 | 1 | -0/+393 |
| * | | ci/request-reviews: Request reviews for individual team members•••This makes this codeowner mechanism behave differently than the native
one, but there's no other way to avoid rerequesting reviews from teams
when a member already reviewed the PR.
| Silvan Mosberger | 2024-10-09 | 1 | -8/+30 |
| * | | ci/request-reviews: Don't rerequest users that already reviewed•••The automation should never rerequest reviews from users that already
reviewed the changes, which is what was happening before this change:
https://github.com/NixOS/nixpkgs/pull/347354#event-14570645380
Also reorder the arguments to make more sense
| Silvan Mosberger | 2024-10-09 | 2 | -7/+23 |
| |/ |
|
| * | ci: Add codeowners validator | Silvan Mosberger | 2024-10-08 | 4 | -0/+83 |
| * | ci: Add review request scripts•••Also post a comment in case base branch is wrong
This guides newcomers in how to smoothly handle the potentially scary
situation of having thousands of commits listed in a PR.
While CI shows the same, people might not even look at CI if the PR
looks botched.
| Silvan Mosberger | 2024-10-08 | 6 | -0/+338 |
| * | ci: Add default.nix with exposed pkgs•••Allows reusing it in more places
| Silvan Mosberger | 2024-10-08 | 1 | -0/+27 |
| * | treewide: Fix or remove some markdown links•••Found using https://github.com/serokell/xrefcheck, which unfortunately
can't trivially be enforced in CI because we also have the manual markdown
files that need post-processing to be valid
| Silvan Mosberger | 2024-09-03 | 1 | -1/+1 |
| * | nixpkgs-vet: update CI, docs, and release to 0.1.4•••Everything gets moved into the `ci/` top-level directory.
We keep behind `maintainers/scripts/check-by-name.sh` and `pkgs/test/check-by-name/pinned-version.txt` as they are going to cause CI errors and confusion until we get all the way through the various channels.
They'll be removed in about a week or so.
| Philip Taron | 2024-09-03 | 4 | -0/+125 |
| * | ci: Update pinned Nixpkgs•••From the nixpkgs-unstable channel: https://hydra.nixos.org/eval/1808410#tabs-inputs
| Silvan Mosberger | 2024-08-22 | 1 | -2/+2 |
| * | ci/pinned-nixpkgs.json: update•••Update it to https://hydra.nixos.org/eval/1807730#tabs-inputs, which
notably contains some fixes for nixfmt: https://github.com/NixOS/nixfmt/pull/215
| Silvan Mosberger | 2024-07-18 | 1 | -2/+2 |
| * | ci/update-pinned-nixpkgs.sh: Allow setting the rev•••This script only needs to make sure that the revision has the relevant
packages built, it doesn't necessarily need to be a channel version.
This commit makes it possible to set the revision explicitly when
calling the update script.
| Silvan Mosberger | 2024-07-18 | 1 | -1/+2 |
| * | ci: reusable Nix format Nixpkgs pin•••This is needed such that in the next commit, we can re-use the same
version from a shell.nix, allowing people to have a guaranteed matching
nixfmt version.
| Silvan Mosberger | 2024-06-26 | 3 | -0/+28 |
| * | ci: init directory•••This creates a new directory for all CI support files, will be populated
in future commits.
| Silvan Mosberger | 2024-06-26 | 1 | -0/+4 |
