Ability to opt-in pull requests/packages to run check in CI #20675
Replies: 4 comments 1 reply
-
Warning: This change would break every assumptions from ArchLinux (PKGBUILD), AlpineLinux (APKBUILD) etc. packaging system. |
Beta Was this translation helpful? Give feedback.
-
@jannick0 suggested in #19545 to run check in contributors forks. |
Beta Was this translation helpful? Give feedback.
-
Do we want to run checks after merging in master ? Solutions to that problem include:
|
Beta Was this translation helpful? Give feedback.
-
I created a PR in my fork to test @jannick0's suggestion but CI is not running. |
Beta Was this translation helpful? Give feedback.
-
In #20651, @filnet requested a way to run packages'
check
functions in a PR.Potential ideas for accomplishing this:
_check
or something like that), or the PR has arun-checks
label, which would be cleaner, except it still requires manual intervention by maintainers who have rights to label PRs (you can set labels in an issue template but apparently not in a PR template).ci-dont-install-list.txt
to determine whether or not to run checks for a given package. I am concerned that that does not scale well if the list file gets long, and that not every PR for a package would necessarily want to run the checks. (See 714a25f)I felt this was getting kind of off-topic for the PR in question, and wanted to expand the discussion to hopefully get a feel if there are other ideas, or if any of these ideas are preferred over others. I was kind of partial to triggering off of a PR label, but can't see a way to let unprivileged users use it (without also adding some bot-type thing that users can ask to do it)
Beta Was this translation helpful? Give feedback.
All reactions