summaryrefslogtreecommitdiff
path: root/.github (follow)
Commit message (Expand)AuthorAgeFilesLines
* .github/labeler.yml: label cinnamon itself (#431750)Bobby Rong2025-08-071-0/+1
|\
| * .github/labeler.yml: label cinnamon itself•••See 2c551ea2d35051b0e4f166b8a7fa659b01205c6c. Bobby Rong2025-08-071-0/+1
* | workflows/eval: disable swap•••Recent performance tests show that (a) swapping heavily slows down the Eval job, while (b) lowering the chunkSize does not have an effect on run-time. It does on memory usage, though - thus we can get rid of swapping entirely by reducing chunkSize respectively. Wolfgang Walther2025-08-061-6/+5
|/
* workflows/build: build nixpkgs tarball•••This adds a build job for the tarball, which might help uncover eval issues on attributes not normally touched by Eval, aka those added in `pkgs/top-level/packages-config.nix`. Wolfgang Walther2025-07-241-2/+6
* .github/PULL_REQUEST_TEMPLATE.md: fix typoMichael Daniels2025-07-201-1/+1
* .github/PULL_REQUEST_TEMPLATE: shorten (#425831)Wolfgang Walther2025-07-181-26/+22
|\
| * .github/PULL_REQUEST_TEMPLATE: shorten TODO list•••This doesn't really change the items, but shortens the writing a lot, making them much more readable, especially while still drafting the PR. Mentioning the release notes for the previous release is not really important, because it doesn't apply for the big majority of pull requests. Wolfgang Walther2025-07-181-15/+15
| * .github/PULL_REQUEST_TEMPLATE: remove mention of linking NixOS tests•••This *is* important, but is a niche detail which belongs into the contributions guidelines - and not into the PR template. It's also out of place for "what did you test?". Wolfgang Walther2025-07-181-2/+0
| * .github/PULL_REQUEST_TEMPLATE: avoid inlining links for readability•••We expect the TODO list to be read through *on PR creation*, otherwise the html comments would not make sense. Thus, we should make it even only slightly readable, which was not at all the case before. The links for release notes are removed, because the PR author has no value from the *current* release notes. They will need to find the file manually anyway. Wolfgang Walther2025-07-181-8/+20
| * .github/PULL_REQUEST_TEMPLATE: remove note about reviews•••While this note is important, it's also mostly invisible at this stage. The comment only shows while creating the PR, but at this stage the author really has other things to worry about. If they care, they will read the contribution guidelines and will pick up the pieces about reviewing that way. If they don't - they won't be bothered by this notice either. Wolfgang Walther2025-07-181-11/+0
| * .github/PULL_REQUEST_TEMPLATE: remove sandbox checkbox•••This checkbox has been used very incosistently and its meaning is not entirely clear: What does it mean if both checkboxes are *unchecked*? Does that mean the sandbox was disabled? Or does it mean the checkbox was just not handled? Also the new nixpkgs-review-gha, which is increasingly used to test builds on darwin platforms shows this information as part of the review - where it's in a much better place. Wolfgang Walther2025-07-181-3/+0
* | ci/github-script/commits: block on errors•••Most of the checks we do for cherry-picks are dismissable warnings, with one exception: When a commit hash has been found, but this hash is not available in any of the pickable branches, we raise this with severity=error. This should also *block* the merge and not be dismissable. That's because this is a fixable issue in every case. Wolfgang Walther2025-07-161-77/+1
|/
* ci/github-script/commits: init from ci/check-cherry-picks•••This turns the check-cherry-pick script into a github-script based JavaScript program. This makes it much easier to extend to check reverts or merge commits later on. Wolfgang Walther2025-07-161-27/+10
* ci/github-script: default to commonjs•••Since all github-scripts need to be written in commonjs, we now default to it by not setting package.json. Support from editors for .js files is slightly better than .cjs. To still allow using module imports in the test runner script, we trick node into loading the script itself as a module again via `--import ./run`. Wolfgang Walther2025-07-141-1/+1
* ci/github-script: move from ci/labels•••This just moves things around to use less specific naming - `labels` is only *one* script that can potentially be run locally while still being written in github-script. Later, we can add more. Wolfgang Walther2025-07-141-2/+2
* workflows/backport: fix concurrent jobs cancelling each other•••When a PR is merged and labeled afterwards - with a non-backport label - the following will happen: - The first backport job is triggered on the merge. - The second backport job is triggered on the label event. - The second job will cancel the first one due to the concurrency group. - The second job will cancel itself because the label event didn't contain a backport label. Both jobs end up cancelled and no backport happens. We made the backport action idempotent upstream a while ago, so we don't need to cancel those actions. Instead, we'll run all of them - subsequent actions running through will just stay silent anyway. Wolfgang Walther2025-07-121-4/+0
* workflows/build: be clearer about what is being built•••Committers could get the false impression from, e.g., `PR / Build / aarch64-linux` that this workflow builds the packages changed in the current PR. Such a misunderstanding could pair poorly with the "enable auto-merge" button, once that's enabled. Michael Daniels2025-07-111-1/+5
* ci/labels: run in dry mode locally•••To avoid mistakes when developing and testing against the upstream repo. Wolfgang Walther2025-07-081-0/+1
* ci/labels: init from workflows/labels•••Moves the labels job into a separate ci/ subfolder to run it locally. This eases debugging *a lot*. Wolfgang Walther2025-07-081-399/+9
* workflows/labels: small refactor•••To avoid having a diff when moving the file in the next commit. Wolfgang Walther2025-07-071-1/+1
* workflows/labels: manage "needs: reviewer" label•••This label allows finding pull requests which have no reviewer requested, yet. Wolfgang Walther2025-07-041-6/+25
* workflows/labels: paginate with cursor•••Pagination via cursor is required above 10k items. To do so, we store the current cursor as an artifact and read it back in in the next scheduled run. Wolfgang Walther2025-07-031-32/+59
* Reapply "workflows/labels: label stale issues"•••This reverts commit c18e94361ef21171c48522142adfbb1081be90bf. Wolfgang Walther2025-07-022-150/+153
* Revert "workflows/labels: label stale issues"Wolfgang Walther2025-07-012-153/+150
* workflows/labels: label stale issues (#419878)Wolfgang Walther2025-07-012-150/+153
|\
| * workflows/labels: label stale issues•••By re-organizing the flow in `handle()` we can start labeling both issues and pull requests, and only make the relevant API requests for the PR-case. At first glance, we might think that we only need to label the big batch list of issues and not those recently updated: But that's wrong, for recently updated issues it's important to label quickly, because the stale label needs to be *removed*, too. Wolfgang Walther2025-06-272-150/+153
* | .github/ISSUE_TEMPLATE: remove 24.11 references•••24.11 is unmaintained now Leona Maroni2025-07-016-6/+0
* | .github/labeler: remove "backport release-24.11" auto label•••24.11 is unmaintained. Leona Maroni2025-07-011-7/+0
* | .github/PULL_RQUEST_TEMPLATE: remove 24.11 references•••24.11 is unmaintained now. Leona Maroni2025-07-011-2/+2
* | workflows/periodic-merge: remove 24.11•••24.11 is deprecated now. Leona Maroni2025-07-011-4/+0
* | .github/workflows/README.md: one sentence per lineWolfgang Walther2025-06-291-13/+34
|/
* workflows/labels: retry on transient API failures•••Currently, the labels job fails a few times each day with network failures. Retrying the requests should help. Wolfgang Walther2025-06-271-0/+1
* workflows/labels: label rebuilds on failed PR workflow•••We already tried to fix this case earlier, but didn't account for all cases: A scheduled workflow can also encounter a pull request with failed PR workflow. This failure doesn't need to be in the Eval part, so artifacts could *still* be available. To make sure PRs always get rebuild labels, just ignore the status condition. Either the artifact is there, or it is not. Wolfgang Walther2025-06-271-2/+0
* workflows/labels: slightly improve loggingWolfgang Walther2025-06-271-0/+1
* workflows/labels: fix processing the 100 oldest PRs•••The `page` number is 1-based, but the remainder might very well be 0. This lead to not looking at the 100 oldest PRs, ever. Wolfgang Walther2025-06-271-1/+1
* .github/labeler: label changes to nixpkgs release notes as "has changelog" (#...Wolfgang Walther2025-06-261-0/+1
|\
| * .github/labeler: label changes to nixpkgs release notes as "has changelog"•••Previously only the NixOS release notes were taken into account. Wolfgang Walther2025-06-261-0/+1
* | workflows/backport: korthout/backport-action: 3.2.0 -> 3.2.1•••Release Notes: https://github.com/korthout/backport-action/releases/tag/v3.2.1 This should many of the annoying, duplicated error messages that the backport action comments. Wolfgang Walther2025-06-261-1/+1
|/
* workflows/labels: fix merge conflict label•••The previous implementation had two problems: - When switching from /search to /pulls, we disabled the additional GET on each single pull request - which causes no test merge commit creation for all PRs. This means, merge conflicts will not actually be detected. - By using `item` in the pull-request triggered case, this goes back to `context.payload.pull_request`, which is the state *at the beginning* of the workflow run. But this renders our "let's wait 3 minutes before checking merge_commit_sha" logic void. While we wait for 3 minutes, we still use the *old* value afterwards... Just making the extra request every time simplifies the logic and solves both problems. Wolfgang Walther2025-06-251-5/+2
* workflows/labels: fix stale label date sorting•••With the help of: https://stackabuse.com/how-to-sort-an-array-by-date-in-javascript/ Wolfgang Walther2025-06-251-2/+2
* Reapply "workflows/labels: manage stale & merge conflict labels" (#419654)Wolfgang Walther2025-06-251-147/+248
|\
| * workflows/labels: use /pulls endpoint instead of search for "all" pull requests•••It's necessary to use a combination of different endpoints here, because the /search endpoint only allows fetching the first 1000 items and will fail with a higher page number (11+). On the flip side, the /pulls endpoint doesn't allow counting the total number of results, so we can't calculate the required page number with its response. Putting both together should work, though. Wolfgang Walther2025-06-251-12/+15
| * workflows/labels: improve cleanup of reservoir timer•••This should make sure that the timer is cleaned up, no matter what. This didn't seem to be the case before, where it would still be stuck sometimes, when throwing an error somewhere. Wolfgang Walther2025-06-251-61/+63
| * workflows/labels: handle PR-creation-edge-case for merge conflict label•••Explained very well by the code comment. Wolfgang Walther2025-06-251-1/+8
| * workflows/labels: fix running in pull_request context•••When running in a pull_request context, the labels job is part of the currently running workflow - which will never have succeeded, yet. Apparently it could be failed already, so in this case we take *any* workflow run, no matter its state. Wolfgang Walther2025-06-241-1/+2
| * workflows/labels: fix stale label•••To set the stale label properly, we need to consider the right timeline events only - and their respective relevant timestamps. Wolfgang Walther2025-06-241-4/+25
| * Reapply "workflows/labels: manage stale & merge conflict labels"•••This reverts commit c366efa6e2816c2cb48b3018fe00ceb8ca6cbc81. Wolfgang Walther2025-06-241-144/+211
* | workflows: nix: 2.29.0 -> 2.29.1Wolfgang Walther2025-06-245-10/+10
|/
* Revert "workflows/labels: manage stale & merge conflict labels"Wolfgang Walther2025-06-241-211/+144
* workflows/labels: manage merge-conflict label for pull requests•••The code comments describe much better what we do then a commit message could ever do. Wolfgang Walther2025-06-241-0/+14