- Issue created by @sunil jolapara
- Merge request !5Resolve #3475445 "Fix the phpcodesniffer issues" → (Closed) created by sunil jolapara
- Status changed to Needs review
7 months ago 7:02am 19 September 2024 - 🇺🇸United States sunil jolapara 3265 Summitrun Drive, Independence, KY, 41051
- 🇮🇹Italy apaderno Brescia, 🇮🇹
The issue summary for PHP_CodeSniffer issues should always show the report given from PHP_CodeSniffer and the arguments passed to
phpcs
.Furthermore, since GitLab CI is now enabled for all the projects, GitLab CI should be used to get PHP_CodeSniffer errors/warnings.
- 🇨🇦Canada mparker17 UTC-4
mparker17 → changed the visibility of the branch 3475445-fix-the-issues to hidden.
- 🇨🇦Canada mparker17 UTC-4
mparker17 → changed the visibility of the branch 3475445-fix-the-issues to active.
- Merge request !6Issue #3475445 by mparker17, avpaderno: Fix the issues reported by PHP_CodeSniffer → (Merged) created by mparker17
- 🇨🇦Canada mparker17 UTC-4
@avpaderno agreed; thank you.
I've created a new branch that only fixes the errors from GitLab CI and hidden the old branch / merge request for this issue.
All tests seem to be passing, so I'm going to merge shortly.
-
mparker17 →
committed fd03496f on 2.0.x
Issue #3475445 by mparker17, avpaderno: Fix the issues reported by...
-
mparker17 →
committed fd03496f on 2.0.x
Automatically closed - issue fixed for 2 weeks with no activity.