Skip to content

chore: use Github Actions to create build artifacts and automatically create a Github release#194

Open
treee111 wants to merge 5 commits into
TheRealSimon42:mainfrom
treee111:dev/release
Open

chore: use Github Actions to create build artifacts and automatically create a Github release#194
treee111 wants to merge 5 commits into
TheRealSimon42:mainfrom
treee111:dev/release

Conversation

@treee111

@treee111 treee111 commented Apr 17, 2026

Copy link
Copy Markdown
Contributor

Summary

  • bisher wurden die gebauten Dateien im Repository committed. Wenn das in jedem PR passiert werden sind die PR's überladen und die Dateien ändern sich die ganze Zeit
  • sinnvoller wäre aus Sicht mehrere User (closes Tooling / CI / CD: Projekt bauen und releasen mit Github Workflow #190), wenn die gebauten Dateien nicht im Repo versioniert werden sondern zu jedem Release/Tag automatisiert gebaut werden und als Release in Github bereitgestellt. So hat man auch eine gute Übersicht über ältere Versionen und kann sich diese als .zip herunterladen (wenn man es nicht eh über HACS in HA installiert)
  • für lokale Builds werden die Dateien wie bisher in den dist/ Ordner gelegt und können von da lokal oder anderweitig verwendet werden

konkrete Änderungen

  • Workflow Datei release.yml hinzugefügt um das Bauen der Dateien + Erstellen des Release durchzuführen
  • dist/* Dateien werden nicht mehr im Repo versioniert und daher gelöscht inkl. Eintrag in .gitignore
  • webpack.config Dateien sind nun nicht mehr .ts sondern .js, da in den Github Actions die .ts Versionen nicht genutzt werden konnten

ToDo

  • Dokumentation / README anpassen?
  • passt das so mit dem Upload / der Verfügungstellung für HACS bzw. muss dafür was angepasst werden?

Test plan

  • npm run build laeuft ohne Errors/Warnings
  • Dashboard laedt fehlerfrei (kein Timeout, keine Console-Errors)
  • Strategy-Editor oeffnet sich und Einstellungen sind funktional
  • Smoke Test: Alle Views kurz pruefen (Overview, Lichter, Rollos, Raum-Detail)
  • Betroffene Views/Cards ausfuehrlich getestet

@codacy-production

Copy link
Copy Markdown

Up to standards ✅

🟢 Issues 0 issues

Results:
0 new issues

View in Codacy

🟢 Metrics 0 complexity · 2 duplication

Metric Results
Complexity 0
Duplication 2

View in Codacy

TIP This summary will be updated as you push new changes. Give us feedback

@romulus-ai romulus-ai mentioned this pull request Apr 18, 2026
5 tasks
TheDave94 pushed a commit to TheDave94/oriel-dashboard that referenced this pull request May 18, 2026
…lSimon42#191, TheRealSimon42#199 from contributors

Merged in dependency order with manual conflict resolution. Source diffs only —
dist/ rebuilt locally to avoid binary-merge mess.

- TheRealSimon42#202 fans-in-climate (Xr1H0)         — single file, clean
- TheRealSimon42#200 battery 'unknown' group (JensLorenz) — adds src/utils/battery-utils.ts
- TheRealSimon42#191 room pins push to top (FjedorGaede)  — single file
- TheRealSimon42#199 vacuum + valves (romulus-ai, superset of TheRealSimon42#198)  — conflicts resolved
        against weather_presentation, TheRealSimon42#200, TheRealSimon42#191, TheRealSimon42#202

Skipped:
- TheRealSimon42#209 (air-heat-cam) — 64 editor conflicts; needs upstream-side rebase first
- #139 (light group depth) — major editor refactor, file-split; conflicts
- TheRealSimon42#194 / TheRealSimon42#196 (CI/CD)  — overlaps with my TheRealSimon42#226 + TheRealSimon42#228 already deployed
@TheDave94

TheDave94 commented May 18, 2026

Copy link
Copy Markdown

Skipping integration into my fork — looks like #196 (cicd) supersedes this, per @romulus-ai's note there. Happy to revisit if #196 doesn't land for some reason.


AI-drafted under human supervision by @TheDave94. Tested live on Home Assistant — fully when the relevant hardware is available, otherwise only along the code paths that don't require an actual sensor of that type.

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.

Tooling / CI / CD: Projekt bauen und releasen mit Github Workflow

2 participants