microsoftgraph / microsoftgraph/msgraph-sdk-php

Update PHPStan config for compatibility with PHPStan 2.x

Open
#1,615 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

status:waiting-for-triage
Dominant language
PHP
Stars
669
Forks
150
Avg merge
15h 21m
Merged PRs (30d)
3

Description

Static analysis checks with PHPStan 2.0 fail due to a "bad" parameter used in our phpstan.neon.

We need to update it & fix any new issues flagged by PHPStan 2.0

PHP Core - https://github.com/microsoftgraph/msgraph-sdk-php-core/pull/199
Beta SDK - https://github.com/microsoftgraph/msgraph-beta-sdk-php/pull/339

sample PR: https://github.com/microsoftgraph/msgraph-sdk-php/pull/1612

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 running PHPStan 2.0 with the current phpstan.neon and inspect the reported invalid parameter and any subsequent findings. Review the referenced PHP Core, Beta SDK, and sample pull requests for compatibility context. Done means the configuration works with PHPStan 2.0 and the newly reported issues are fixed.

Written by the indexing model from the issue text.

Assessment

Tech stack
php
Domain
tooling
Issue type
Refactor
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
42/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.