GH2863: Add GitHub Actions workflow commands#4033
GH2863: Add GitHub Actions workflow commands#4033devlead merged 2 commits intocake-build:developfrom
Conversation
|
Could you add some integration tests here: Wonder if group / endgroup should have an IDisposable helper so one could do a using statement like TravisCI fold That could be in follow up PR tomorrow if your short in time. |
|
It might take a couple of iterations to get the integration tests working satisfactorily. Bear with me. |
|
@devlead I see the group but not the message inside the group for some reason. Is this something to do with how the integration tests are run? What am I missing? |
19ea230 to
1d8a35c
Compare
Think it needs to be quoted, example of the command used in wild if (BuildSystem.GitHubActions.IsRunningOnGitHubActions)
{
TaskSetup(context=> System.Console.WriteLine($"::group::{context.Task.Name.Quote()}"));
TaskTeardown(context=>System.Console.WriteLine("::endgroup::"));
}Result: |
|
Hmm also looks like |
|
No, doesn't need to be quoted, and that output is from the log which seems to be related to azure pipelines legacy. Basically the group is working fine. It's the following information output that's missing. Same problem with the secret test. I've been using (unquoted) groups for a while: https://github.com/gitfool/Cake.Dungeon/blob/3d9f8c6fd79cb74089e7f4d6d70c68fda5c20662/scripts/bootstrap.cake#L24-L38 |
|
Annotations are working as expected. It's only the |
Ah, the information you won't see as the integration tests run at a lower verbosity. I switched to a |
|
@gitfool your changes have been merged, thanks for your contribution 👍 |



Fixes #2863.
Note: implementation guided by looking at:
@devlead I'd love to squeeze in this pull request just in time for Hacktoberfest. 😉