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

use rex-cs-fixer-config 2.x #55

Open
staabm opened this issue Apr 2, 2023 · 3 comments
Open

use rex-cs-fixer-config 2.x #55

staabm opened this issue Apr 2, 2023 · 3 comments
Milestone

Comments

@staabm
Copy link
Member

staabm commented Apr 2, 2023

Feature description / Feature Beschreibung
nachdem die php/redaxo-core mindestversion angehoben wäre könnte man cs-fixer-config 2.x verwenden

refs redaxo/php-cs-fixer-config#5

@staabm staabm added this to the next-major milestone Apr 2, 2023
@aeberhard
Copy link
Member

Könnte man nicht beide einbinden? Ich verwende z.b. im VSCode den 2.1.1

  • REDAXO specific code style v1
  • REDAXO specific code style v2
  • More explicit coding style

@aeberhard
Copy link
Member

@staabm hier sollte auf jeden Fall V2 verwendet werden :)

@staabm
Copy link
Member Author

staabm commented Oct 8, 2023

Würde bedeuten dass wir die php mindestversion von rexfactor auf 8.1 anheben müssten. Ich weiß noch nicht ob das eine gute idee wäre. Aber als PR könntest du es definitiv vorbereiten :-)

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

No branches or pull requests

2 participants