wordpress-mobile / release-toolkit-gutenberg-mobile

Automation Scripts for Releasing Gutenberg-Mobile Updates to the WordPress Mobile Apps.
Mozilla Public License 2.0
5 stars 2 forks source link

fix: iOS update ignores trailing comma #140

Closed dcalhoun closed 1 year ago

dcalhoun commented 1 year ago

Ignoring the trailing comma makes this script more resilient.

Including the trailing comma in the matcher, but not the replacement, resulted in removing the trailing comma. This meant the next run would fail as the trailing comma would no longer be present. Also, the trailing comma may be removed by chance when the commit is manually updated during normal development, which can occur and is quite harmless.

The original matcher was created in dc44c49. Examples of the trailing comment changing can be found in the following: