add group and endgroup commands

[thboop]

Tested on the following yaml:

on: push
name: Test
jobs:
  abbbbbb:
    name: abbbbbb
    runs-on:
      pool: default
    steps:
    - name: aaaaaaa
      id: step1
      run: |
        echo ::group::
        echo hello world
        echo test
        echo ::endgroup::
        echo test
        echo test2

got the following output

2019-09-16T20:53:12.1180421Z ##[group]Run echo ::group::
2019-09-16T20:53:12.1230404Z �[36;1mecho ::group::�[0m
2019-09-16T20:53:12.1230680Z �[36;1mecho hello world�[0m
2019-09-16T20:53:12.1230885Z �[36;1mecho test�[0m
2019-09-16T20:53:12.1231089Z �[36;1mecho ::endgroup::�[0m
2019-09-16T20:53:12.1231293Z �[36;1mecho test�[0m
2019-09-16T20:53:12.1231492Z �[36;1mecho test2�[0m
2019-09-16T20:53:12.1237271Z shell: C:\windows\system32\cmd.exe /D /E:ON /V:OFF /S /C "CALL "{0}""
2019-09-16T20:53:12.1238121Z ##[endgroup]
2019-09-16T20:53:12.1600320Z ##[group]
2019-09-16T20:53:12.1601035Z hello world
2019-09-16T20:53:12.1601274Z test
2019-09-16T20:53:12.1601628Z ##[endgroup]
2019-09-16T20:53:12.1601862Z test
2019-09-16T20:53:12.1602130Z test2

[bryanmacfarlane]

Shouldn't group have a group title? e.g. echo ::group::foo

[bryanmacfarlane]

^^ https://github.com/actions/toolkit/blob/master/packages/core/src/core.ts#L150

[bryanmacfarlane]

Also, the ::group shouldn't make it's way to the log / server, right? The step should emit the consistent ::group, the runner intercepts and that turns into ## log decorations, right? In other words, the ::group is the use facing command and the ## is the internal protocol for the log to decorate.

I ask because in the output example above, I see both.

[thboop]

Also, the ::group shouldn't make it's way to the log / server, right? The step should emit the consistent ::group, the runner intercepts and that turns into ## log decorations, right? In other words, the ::group is the use facing command and the ## is the internal protocol for the log to decorate.

I ask because in the output example above, I see both.

It will not, it only did in this example because i'm using a step to echo that value

[thboop]

Shouldn't group have a group title? e.g. echo ::group::foo

this has been updated!

[bryanmacfarlane]

Here's the toolkit issue we can reference in the release: actions/toolkit#112