Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Integrate github workflows #121

Open
wants to merge 8 commits into
base: master
Choose a base branch
from
Open

Conversation

marcohanke
Copy link
Member

No description provided.

Copy link
Member

@christophboecker christophboecker left a comment

Choose a reason for hiding this comment

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

Danke für Deinen Einsatz, Marko! Nur leider kann ich nicht beurteilen, ob das so auch ok ist. Ist halt irgendwie Github-Voodoo und da bin ich dann raus.

Die Anpassungen durch php-cs-fixer verstehe ich noch (nutze ich ja mittlerweile auch im Editor) und das passt ja.

phpunit - keine Ahnung, was ist das?

rexstan - lokal und für neuere Projekte nutze ich es al gusto. Hier wirft es halt massenhaft Fehler raus. Solange das nicht stört und Mergen weiterhin möglich ist, kann kann der Job ruhig drin bleiben.

@marcohanke
Copy link
Member Author

Moin Christoph,
also pass auf. PHPCSfixer verändert Dateien, das kannst du nachvollziehen.
Unittest und RexStan geben dir nur Infos und sie fassen den Code nicht an. Es kann hier also nichts passieren.

Hier findest du weitere Infos:
github.com/friendsOfREDAXO/github-workflows
und ich erkläre es dir gerne noch mal per PN

Aber das wichtigste (abgesehen vom CS Fixer) passiert mit dem Code nichts.
Ich erkläre dir gerne noch mal per PN wie das läuft

@christophboecker
Copy link
Member

Ich verstehe so langsam wie die Sache funktioniert.

  • PHP-CS-Fixer: check
  • PHPUnit: kann man so lassen; stört nicht. Um umfassend zu testen müsste ich vermutlich das X-fache der Entwicklungszeit noch mal reinstecken (x größer 1).
  • -rexstan: da schaue ich mir die .tools/rexstan.php noch mal näher an. Ich will die help.php vom Scan ausschließen. Mit Rexstan-Bordmitteln ist die Änderung der user-config.neon arg eingeschränkt. Aber das krieg ich hin :-)

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.

2 participants