Skip to content

Commit 367377e

Browse files
ap0llonils-a
authored andcommitted
Make the section header more descriptive
Instead of just using the task name as section header, use "Executing task TASKNAME". This makes the section headers more consistent with the section headers used by GitLab CI which uses e.g. "Executing "step_script" stage of the job script"
1 parent 3141f75 commit 367377e

File tree

1 file changed

+7
-7
lines changed

1 file changed

+7
-7
lines changed

src/Cake.GitLabCI.Module/GitLabCIEngine.cs

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -45,17 +45,17 @@ public GitLabCIEngine(ICakeDataService dataService, ICakeLog log, IConsole conso
4545

4646
private void OnBeforeSetup(object sender, BeforeSetupEventArgs e)
4747
{
48-
WriteSectionStart("Setup");
48+
WriteSectionStart("setup", "Executing Setup");
4949
}
5050

5151
private void OnAfterSetup(object sender, AfterSetupEventArgs e)
5252
{
53-
WriteSectionEnd("Setup");
53+
WriteSectionEnd("setup");
5454
}
5555

5656
private void OnBeforeTaskSetup(object sender, BeforeTaskSetupEventArgs e)
5757
{
58-
WriteSectionStart(GetSectionNameForTask(e.TaskSetupContext.Task.Name), e.TaskSetupContext.Task.Name);
58+
WriteSectionStart(GetSectionNameForTask(e.TaskSetupContext.Task.Name), $"Executing task \"{e.TaskSetupContext.Task.Name}\"");
5959
}
6060

6161
private void OnAfterTaskTeardown(object sender, AfterTaskTeardownEventArgs e)
@@ -65,17 +65,17 @@ private void OnAfterTaskTeardown(object sender, AfterTaskTeardownEventArgs e)
6565

6666
private void OnBeforeTeardown(object sender, BeforeTeardownEventArgs e)
6767
{
68-
WriteSectionStart("Teardown");
68+
WriteSectionStart("teardown", "Executing Teardown");
6969
}
7070

7171
private void OnAfterTeardown(object sender, AfterTeardownEventArgs e)
7272
{
73-
WriteSectionEnd("Teardown");
73+
WriteSectionEnd("teardown");
7474
}
7575

76-
private void WriteSectionStart(string sectionName, string sectionHeader = null)
76+
private void WriteSectionStart(string sectionName, string sectionHeader)
7777
{
78-
_console.WriteLine("{0}", $"{AnsiEscapeCodes.SectionMarker}section_start:{DateTimeOffset.UtcNow.ToUnixTimeSeconds()}:{sectionName}\r{AnsiEscapeCodes.SectionMarker}{AnsiEscapeCodes.ForegroundBlue}{sectionHeader ?? sectionName}{AnsiEscapeCodes.Reset}");
78+
_console.WriteLine("{0}", $"{AnsiEscapeCodes.SectionMarker}section_start:{DateTimeOffset.UtcNow.ToUnixTimeSeconds()}:{sectionName}\r{AnsiEscapeCodes.SectionMarker}{AnsiEscapeCodes.ForegroundBlue}{sectionHeader}{AnsiEscapeCodes.Reset}");
7979
}
8080

8181
private void WriteSectionEnd(string sectionName)

0 commit comments

Comments
 (0)