🐛 Explicit wrap messages with quotes

[segersniels]

Description

Fixes #1391. Not sure if this ever reported with quotes around the message but execa seems to automatically escape quotes and at one point it stopped working.

Added

• Added shell: true option to git commit execution

Changed

• Updated commit message formatting to include quotes around both title and message parts
• Modified test stubs and expectations to match the new quoted format

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 92.43%. Comparing base (8435dd4) to head (1349714).
Report is 1 commits behind head on master.

Additional details and impacted files

@@           Coverage Diff           @@
##           master    #1392   +/-   ##
=======================================
  Coverage   92.43%   92.43%           
=======================================
  Files          29       29           
  Lines         291      291           
  Branches       66       66           
=======================================
  Hits          269      269           
  Misses         21       21           
  Partials        1        1           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[segersniels]

Not a fan of using shell as they advise against it in the documentation of execa. https://github.com/sindresorhus/execa/blob/99d1741d2525eca71b986282148bbf2983356428/docs/shell.md#avoiding-shells

[carloscuesta]

Seems that somehow now they do escape " the only option mentioned in the docs is using shell 😅

https://github.com/sindresorhus/execa/blob/main/docs/escaping.md#array-syntax

[segersniels]

Seems that somehow now they do escape " the only option mentioned in the docs is using shell 😅

https://github.com/sindresorhus/execa/blob/main/docs/escaping.md#array-syntax

I suppose we can merge it and revert once people start running into issues due to now configuring shell.

[carloscuesta]

Let's see @segersniels just released this