Skip to content

chore: update psalm baseline.xml#7516

Merged
kenjis merged 3 commits intocodeigniter4:developfrom
kenjis:update-psalm-baseline.xml
May 24, 2023
Merged

chore: update psalm baseline.xml#7516
kenjis merged 3 commits intocodeigniter4:developfrom
kenjis:update-psalm-baseline.xml

Conversation

@kenjis
Copy link
Copy Markdown
Member

@kenjis kenjis commented May 23, 2023

Description
To suppress the following error and warnings:

ERROR: UnsupportedPropertyReferenceUsage - system/Test/ControllerResponse.php:51:9 - This reference cannot be analyzed by Psalm. (see https://psalm.dev/321)
        $this->dom = &$this->domParser;

ControllerResponse is deprecated.

Warning: "findUnusedBaselineEntry" will be defaulted to "true" in Psalm 6. You should explicitly enable or disable this setting.
Warning: "findUnusedCode" will be defaulted to "true" in Psalm 6. You should explicitly enable or disable this setting.

Checklist:

  • Securely signed commits
  • [] Component(s) with PHPDoc blocks, only if necessary or adds value
  • [] Unit testing, with >80% coverage
  • [] User guide updated
  • Conforms to style guide

kenjis added 3 commits May 24, 2023 08:18
To suppress the following warnings:
Warning: "findUnusedBaselineEntry" will be defaulted to "true" in Psalm 6. You should explicitly enable or disable this setting.
Warning: "findUnusedCode" will be defaulted to "true" in Psalm 6. You should explicitly enable or disable this setting.

findUnusedBaselineEntry="false"
Because enabled PHP extensions vary depending on the developer's environment,
setting this to true will likely result in errors.
@kenjis kenjis force-pushed the update-psalm-baseline.xml branch from e4dcecb to 77b0b1d Compare May 23, 2023 23:37
@kenjis kenjis added github_actions Pull requests that update Github_actions code and removed github_actions Pull requests that update Github_actions code labels May 23, 2023
@kenjis kenjis merged commit c33dd88 into codeigniter4:develop May 24, 2023
@kenjis kenjis deleted the update-psalm-baseline.xml branch May 24, 2023 08:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants