owncloud / owncloud/QA

Create Phan / PHPSTAN extension to replace occ app:check-code

Open
#587 2 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

dev:code-analysis QA-p4 QA:team
Dominant language
Shell
Stars
16
Forks
16
Avg merge
11h 11m
Merged PRs (30d)
1

Description

The code-checker inside ownCloud has been useful in the past. Instead of however needing to maintain (and ship it as supportable code), lets move the functionality to a static code analyzer like phan or phpstan

Code in question:
https://github.com/owncloud/core/tree/master/lib/private/App/CodeChecker

cc @PVince81 @DeepDiver1975

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start by reviewing the existing implementation in lib/private/App/CodeChecker and compare its behavior with the proposed Phan or PHPStan extension. Identify which app:check-code functionality must be preserved and how the replacement would be integrated. Done means the existing checker functionality is covered by a maintainable static-analysis extension and the old code can be removed or replaced.

Written by the indexing model from the issue text.

Assessment

Tech stack
php
Domain
testing-qa, tooling
Issue type
Feature
Difficulty
5/5
Estimated time
Over a week
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
25/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.