Trailing comma in ::set-output`

[hvr]

I'm not sure if this is relevant, but I noticed that the log shows the meta-command

::set-output name=cache-hit,::true

being emitted, whereas based on Github actions documentation I would have expected

::set-output name=cache-hit::true

Sorry for the noise if this is in fact harmless.

[joshmgross]

All "Commands" issued via the toolkit have this issue, from how they append properties: https://github.com/actions/toolkit/blob/master/packages/core/src/command.ts#L60

Both work, but it does look odd.

Going to transfer this issue to https://github.com/actions/toolkit

[bryanmacfarlane]

Yeah, this is as designed as the , is just an empty property. Both work.

[ericsciple]

the trailing comma is kind of silly, we should probably remove it