Skip to content

Iterate release workflow#53

Merged
awesomephant merged 8 commits into
mainfrom
fix-51
May 26, 2025
Merged

Iterate release workflow#53
awesomephant merged 8 commits into
mainfrom
fix-51

Conversation

@awesomephant
Copy link
Copy Markdown
Collaborator

@awesomephant awesomephant commented May 25, 2025

  • Unser bestehender Release-Workflow folgt nicht den Empfehlungen für Svelte-Bibliotheken
  • Das hat bisher nicht zu Problemen geführt, aber jetzt schon weil wir mehr Typescript Code schreiben, der in unserem Release-Workflow nicht korrekt transpiliert wird und dann in Anwendungen nicht funktioniert
  • Außerdem liefern wir eine Menge Code aus, den Nutzer der Bibliothek nicht wirklich brauchen (zb. unsere Storybook-Konfiguration)

Dieser PR installiert und konfiguriert svelte-package und ändert das Release-Ziel auf components/dist um das zu beheben.

Closes #51

@awesomephant awesomephant linked an issue May 25, 2025 that may be closed by this pull request
Copy link
Copy Markdown
Contributor

@sjockers sjockers left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sieht gut aus!

Was mir aufgefallen ist: Es sieht aus also ob sync immer durchläuft, auch bei einem Fehler (echo ''). Wenn das nötig ist, weil wir Fälle erwarten in denen sync fehlschlägt aber wir trotzdem weitere Skripte ausführen wollen, dann okay. Ansonsten macht es evtl. Sinn das rauszunehmen?

So oder so, kann von mir aus raus.

Comment thread components/package.json Outdated
@awesomephant awesomephant merged commit db84e6d into main May 26, 2025
1 check passed
@awesomephant awesomephant deleted the fix-51 branch May 26, 2025 18:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Fix packaging setup

2 participants