Skip to content

installer_ignore: Entwicklungs- und Test-Artefakte nicht mit ausliefern - #2

Merged
skerbis merged 1 commit into
mainfrom
chore/installer-ignore
Oct 6, 2026
Merged

skerbis merged 1 commit into
mainfrom
chore/installer-ignore

Conversation

@skerbis

@skerbis skerbis commented Sep 23, 2026

Copy link
Copy Markdown
Member

Das Release-ZIP enthielt bisher tests/, phpunit.xml.dist, phpstan.neon.dist und composer.json/composer.lock. Zur Laufzeit wird davon nichts gebraucht: REDAXO registriert für den Autoloader ausschließlich lib/ und vendor/ (rex_package::enlist()), und kein Addon-Code liest diese Dateien.

Warum installer_ignore und nicht .gitattributes

export-ignore greift hier nicht: Die installer-action packt das ZIP mit archiver über einen Glob, nicht mit git archive – .gitattributes wird dabei sogar selbst aus dem Paket entfernt. Der vorgesehene Weg ist installer_ignore in der package.yml; die Action liest den Key, und REDAXOs eigener Installer wertet ihn ebenfalls aus (api_package_upload.php).

README.md bleibt drin – pages/docs.php rendert es als Doku-Seite im Backend.

Geprüft

  • package.yml parst sauber, package/version unverändert.
  • REDAXO liest den Key: rex_addon::get(contacts)->getProperty(installer_ignore) liefert die 8 Einträge (nach cache:clear).
  • Paketierung simuliert (skip-Liste inkl. der Glob-Expansion, die die Action vornimmt): raus fliegen nur die Dev-Artefakte, drin bleiben assets, boot.php, fragments, install.php, lang, lib, modules, pages, uninstall.php, update.php, vendor, ytemplates, README.md, CHANGELOG.md, LICENSE.
  • Backend HTTP 200, keine neuen Einträge im system.log.

🤖 Generated with Claude Code

Das Release-ZIP enthielt bisher tests/, phpunit.xml.dist, phpstan.neon.dist und
composer.json/lock. Zur Laufzeit wird davon nichts gebraucht - REDAXO autoloadet
nur lib/ und vendor/, gelesen wird keine dieser Dateien.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
@skerbis
skerbis merged commit 2178a78 into main Oct 6, 2026
3 checks passed
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.

1 participant