Skip to content

Commit e6ae250

Browse files
fix: failing sync icons workflow due to body-max-line-limit from commitlint (#1831)
* fix: failing sync icons workflow due to body-max-line-limit from commitlint * fix: added changeset --------- Co-authored-by: Zach Davis <25305144+zmdavis@users.noreply.github.com>
1 parent 5b08d61 commit e6ae250

File tree

2 files changed

+11
-4
lines changed

2 files changed

+11
-4
lines changed

.changeset/cute-llamas-kiss.md

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,5 @@
1+
---
2+
"@launchpad-ui/icons": minor
3+
---
4+
5+
Changed the format of the changeset summary when running sync icons workflow manually

packages/icons/scripts/figma-sync.ts

Lines changed: 6 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -402,15 +402,17 @@ async function main(): Promise<void> {
402402
const removed = existingIcons.filter((n) => !upcomingSet.has(n));
403403

404404
if (added.length) {
405-
const addedSummary = `New icons: ${added.join(', ')}`;
405+
const addedList = added.map((name) => ` - ${name}`).join('\n');
406+
const addedSummary = `New icons:\n${addedList}`;
406407
changesetSummary += `\n${addedSummary}`;
407-
log('added', `🆕 ${addedSummary}`);
408+
log('added', `🆕 New icons: ${added.join(', ')}`);
408409
}
409410

410411
if (removed.length) {
411-
const removedSummary = `Removed icons: ${removed.join(', ')}`;
412+
const removedList = removed.map((name) => ` - ${name}`).join('\n');
413+
const removedSummary = `Removed icons:\n${removedList}`;
412414
changesetSummary += `\n${removedSummary}`;
413-
log('removed', `✂️ ${removedSummary}`);
415+
log('removed', `✂️ Removed icons: ${removed.join(', ')}`);
414416
}
415417
} else {
416418
changesetSummary += '\ninitial sync of icons from Figma';

0 commit comments

Comments
 (0)