- Issue created by @Peter Majmesku
- 🇩🇪Germany Hydra
+1 For that in general
Working on a major UI update right now, will look into this afterwords.
- Merge request !36Issue #3496183: Initial commit for the static code analysis → (Merged) created by Peter Majmesku
- 🇩🇪Germany Peter Majmesku 🇩🇪Düsseldorf
I've implemented the GitLab CI pipeline configuration. It handles the static code analysis via PHP_CodeSniffer and PHPStan. This required some code and documentation changes to comply with the standards. All pipelines now pass. The code validation jobs are allowed to fail; this is the default behavior. I recommend always making sure that the jobs succeed. This essentially eliminates all syntax, spelling, and coding standard violations. Please review.
- 🇩🇪Germany Hydra
@peter majmesku Hey thx for the work! Appreciated! I'd like to take a deeper look into this next week with my team
- 🇩🇪Germany Peter Majmesku 🇩🇪Düsseldorf
@hydra: Merged in the current 1.x branch and made the pipelines pass. Please review.
-
hydra →
committed e06ec402 on 1.x authored by
peter majmesku →
Issue #3496183 by peter majmesku: Static code analysis
-
hydra →
committed e06ec402 on 1.x authored by
peter majmesku →
- 🇩🇪Germany Hydra
Hey @peter majmesku! Thx for the work on this. This is a great starting point that will help us continually improving the module, really appreciated!
Automatically closed - issue fixed for 2 weeks with no activity.