Add static code analysis and including CI/CD check
Dieses Issue hat noch niemand übernommen.
Bewertung
- Schwierigkeit
- 4/5
- Geschätzter Aufwand
- 3-5 Tage
- Anfängerfreundlichkeit
- 35/100
- Issue-Typ
- Feature
- Klarheit
- Größtenteils klar
- Aktivitätsstatus
- Veraltet
- Tech-Stack
- github-actions, php
- Bereich
- build-system, ci-cd, tooling
Rechercherichtung
Compare the static-analysis setup in open-runtimes/executor, then inspect the Composer configuration and PHP source directories in utopia-php/cache, utopia-php/platform, and utopia-php/dsn. Start by running the existing checks and progressively validate PHPStan configuration toward level 8. Done means each repository has a check command and a code-analysis.yml workflow that runs composer check on every push.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Beschreibung
Introduction
Static code analysis allows programmers to check their code for bugs ahead of time. There are many ways to do it, but this one specifically understands code without running it, allowing for very fast and reliable checks.
Starting this issue requires an understanding of PHP syntax. Finishing this issue will give you basic understanding about Composer (PHP package manager) and GitHub Actions.
Task summary
Your goal is to implement static code analysis in the following repositories:
1. Install and configure PHPStan
Install PHPStan by using Composer, a PHP dependency manager.
Make sure the version for this dependency is set to automatically upgrade minor and patch versions, but not the major version.
If those terms seem new to you, check out Semantic Versioning.
To configure PHPStan, you use a specific JSON file. Please refer to the PHPStan documentation to learn more.
Your goal is to configure PHPStan in all repositories to the same settings. Please set it to the same settings as we used in another one of our open-source repositories open-runtimes/executor.
In rare scenarios, PHPStan can have issues seeing typings for PHP extensions such as Swoole or Redis. If this happens to you, you might need to mention their helpers in the scanDirectories of the PHPStan configuration.
2. Add command to run static code analysis
Set up Composer scripts called check. This command should run PHPStan on all directories including PHP files. We usually only store PHP code in directories src, app and tests, but make sure to double check it.
When running the PHPStan check, we strive for level 8 of strictness. We recommend you to start with low level of 1, and work your way up, inspecting new errors with each higher level.
If you face issue you can't find a way to solve, please discuss with us and in rare scenarios we allow ignoring a line of code in code analysis.
If you ever get stuck, you can again refer to open-runtimes/executor, to see how it's been done there.
3. Add GitHub automation
Add GitHub action to run linter (composer check) on each push, so pull requests run linter as check.
Store the new GitHub action in the code-analysis.yml file.
Just like before, if you feel stuck, you can refer to Open Runtimes Executor to validate your approach.
- Vorherrschende Sprache
- PHP
- Sterne
- 33
- Forks
- 23
- PR-Merge-Kennzahlen
- Keine gemergten PRs in 30 T.
Entwicklungsumgebung
- Enthält ein Dockerfile oder eine Docker-Compose-Datei
- Keine Pull-Request-Vorlage
- Kein Beitragsleitfaden
Erste Schritte
- Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
- Forken Sie das Repository und arbeiten Sie in einem Branch.
- Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.
Ähnliche Issues
-
FrameworkBundle SecurityBundle
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 68/100
symfony/symfony-docs#23277 ·
Maintainer antworten meist innerhalb von 3 Tagen
-
[Sync EN] hard link does not need special permissionsEvtl. vergeben @lacatoire hat das heute übernommen. Offensync-en
Schwierigkeit 1/5 Unter einer Stunde Anfängerfreundlichkeit 90/100
Maintainer antworten meist innerhalb von 1 Tag
-
sync-en
Schwierigkeit 1/5 Unter einer Stunde Anfängerfreundlichkeit 76/100
Maintainer antworten meist innerhalb von 4 Tagen
-
feature-request
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 72/100
omegaup/omegaup#10323 · 1 Kommentar ·
Maintainer antworten meist innerhalb von 5 Tagen
-
bug mod:appointments
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 78/100
3pontos-tech/gil-benefits#303 ·