Split commands.yml into running and saving
This PR restructures the GH actions into two jobs. First one runs and executes arbitrary code, and is by nature unsafe. It ends with creating a diff patch, and does not have any write permissions to anything. The second job is not user extensible, and applies the diff to the source branch of the PR.
The following comments in a PR can be used as commands to execute scripts which automate repository maintenance and make it part of the visible diff.
/run fantomasrunsdotnet fantomas ./run ilverifyupdates IL verification baseline/run xlfrefreshes localisation files for translatable strings/run test-baseline ...runs tests with theTEST_UPDATE_BSL: 1environment variable and an argument supplied filter (passed todotnet test --filter ..). Its goal is to refresh baselines.
@edgarfgp , @auduchinok : Are there other types of content changes that would also made use of a command, and would not be covered by the above?
:white_check_mark: No release notes required
@T-Gro would be possible to include Re-running the CI ?
Will have to figure out how to do it without a dummy commit (my current naive idea) and without risks with exposing the access token. Will check how this is done elsewhere.
Will have to figure out how to do it without a dummy commit (my current naive idea) and without risks with exposing the access token. Will check how this is done elsewhere.
/azp run will rerun the CI, or was it removed?
/azp runwill rerun the CI, or was it removed?
AFAIK it never worked for OSS contributors, who also do not have the "rerun" buttons when seeing the Checks tab on a PR.
@edgarfgp : What happens when you try /azp run please?
/azp run
Commenter does not have sufficient privileges for PR 18688 in repo dotnet/fsharp
@T-Gro i get Commenter does not have sufficient privileges for PR 18688 in repo dotnet/fsharp
/azp runwill rerun the CI, or was it removed?AFAIK it never worked for OSS contributors, who also do not have the "rerun" buttons when seeing the
Checkstab on a PR.@edgarfgp : What happens when you try
/azp runplease?
Correct, but what if it's person's own PR? I thought arcade folks fixed it - that you can rerun own PR.
/azp runwill rerun the CI, or was it removed?AFAIK it never worked for OSS contributors, who also do not have the "rerun" buttons when seeing the
Checkstab on a PR. @edgarfgp : What happens when you try/azp runplease?Correct, but what if it's person's own PR? I thought arcade folks fixed it - that you can rerun own PR.
Tried in my own PR and did not work. See https://github.com/dotnet/fsharp/pull/18670#issuecomment-2977124473