Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

chore(deps): update dependency vimeo/psalm to v5 #3193

Merged
merged 8 commits into from Feb 4, 2023
Merged

chore(deps): update dependency vimeo/psalm to v5 #3193

merged 8 commits into from Feb 4, 2023

Conversation

renovate[bot]
Copy link
Contributor

@renovate renovate bot commented Nov 30, 2022 •
edited

Mend Renovate

This PR contains the following updates:

Package Type Update Change
vimeo/psalm require-dev major 4.30.0 -> 5.6.0

Release Notes

vimeo/psalm

v5.6.0

Compare Source

What's Changed

Features
Fixes
Typos

New Contributors

Full Changelog: vimeo/psalm@5.5.0...5.6.0

v5.5.0

Compare Source

What's Changed

Features
Fixes
Docs
Internal changes

New Contributors

Full Changelog: vimeo/psalm@5.4.0...5.5.0

v5.4.0

Compare Source

What's Changed

Features
Fixes
Internal changes

Full Changelog: vimeo/psalm@5.3.0...5.4.0

v5.3.0

Compare Source

What's Changed

Features
Fixes
Docs
Internal changes

New Contributors

Full Changelog: vimeo/psalm@5.2.0...5.3.0

v5.2.0

Compare Source

What's Changed

Features
Fixes
Internal changes
Typos

New Contributors

Full Changelog: vimeo/psalm@5.1.0...5.2.0

v5.1.0

Compare Source

What's Changed

Deprecations
Features
Fixes
Docs

New Contributors

Full Changelog: vimeo/psalm@5.0.0...5.1.0

v5.0.0: Psalm 5

Compare Source

Welcome to Psalm 5!

There's an accompanying post on psalm.dev, written by @​muglug & the current maintainers of Psalm.

What's Changed

Removed
Features
Fixes

Configuration

📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).

🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.

♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.

🔕 Ignore: Close this PR and you won't be reminded about this update again.

  • If you want to rebase/retry this PR, check this box

This PR has been generated by Mend Renovate. View repository job log here.

@renovate renovate bot added the renovate label Nov 30, 2022
@renovate renovate bot force-pushed the renovate/major-php-linter branch 3 times, most recently from 359bea5 to 53c79aa Compare December 5, 2022 01:55
@netlify
Copy link

netlify bot commented Dec 5, 2022 •
edited

✅ Deploy Preview for ecamp3-frontend-preview canceled.

Name Link
🔨 Latest commit 5952ce5
🔍 Latest deploy log https://app.netlify.com/sites/ecamp3-frontend-preview/deploys/63d12e64f9e58d00083c9405

@renovate renovate bot force-pushed the renovate/major-php-linter branch 4 times, most recently from 864d4c9 to d126656 Compare December 12, 2022 10:18
@renovate renovate bot changed the title chore(deps): update dependency vimeo/psalm to v5 Update dependency vimeo/psalm to v5 Dec 17, 2022
@renovate renovate bot changed the title Update dependency vimeo/psalm to v5 chore(deps): update dependency vimeo/psalm to v5 Dec 17, 2022
@renovate renovate bot force-pushed the renovate/major-php-linter branch 2 times, most recently from 1d0f3a2 to 5ca7ae2 Compare December 19, 2022 23:25
@renovate renovate bot force-pushed the renovate/major-php-linter branch 17 times, most recently from 4c725f0 to eee21e6 Compare January 5, 2023 02:37
@renovate renovate bot force-pushed the renovate/major-php-linter branch 8 times, most recently from bae9b7e to 42a3be3 Compare January 23, 2023 20:47
@renovate renovate bot force-pushed the renovate/major-php-linter branch 3 times, most recently from 491fdd0 to 5952ce5 Compare January 25, 2023 13:28
@renovate
Copy link
Contributor Author

renovate bot commented Jan 30, 2023

Edited/Blocked Notification

Renovate will not automatically rebase this PR, because it does not recognize the last commit author and assumes somebody else may have edited the PR.
You can manually request rebase by checking the rebase/retry box above.

⚠ Warning: custom changes will be lost.

usu
usu reviewed Feb 2, 2023
View reviewed changes
api/psalm.xml Show resolved Hide resolved
api/src/State/PeriodPersistProcessor.php Show resolved Hide resolved
@BacLuc
Copy link
Contributor

BacLuc commented Feb 4, 2023

@usu with the update of gedmo/sortable doctrine extension in #3244 , i had to add more template parameters:
cc1cbbf

renovate bot and others added 8 commits February 4, 2023 15:14
@renovate @BacLuc
chore(deps): update dependency vimeo/psalm to v5
af50e25
@BacLuc
api/Period: fix Method array_merge called with named unpacked array i…
f8a3626 
…n getContentNodes

https://psalm.dev/268
@BacLuc
api/InvitationProvider: add missing @template-implements
bbecb1c 
App\State\InvitationProvider has missing template params when extending ProviderInterface.
https://psalm.dev/182
@BacLuc
api/PeriodPersistProcessor: fix psalm complaint about possible invali…
d8a3459 
…d offset of array

Cannot access value on variable $days using a int<-1, max> offset, expecting int<0, max>
(see https://psalm.dev/115)
                    $day = $days[$i - 1];

I don't know why line 52 is not a problem...
@BacLuc
api/ResetPasswordProvider: add missing @template-implements
83d3c1b 
ERROR: MissingTemplateParam - src/State/ResetPasswordProvider.php:12:40 - App\State\ResetPasswordProvider has missing template params when extending ApiPlatform\State\ProviderInterface, expecting 1 (see https://psalm.dev/182)
class ResetPasswordProvider implements ProviderInterface {
@BacLuc
api/AssertAllowTransitionsValidator: fix Method array_merge called wi…
6ffe1b4 
…th named unpacked array array

ERROR: NamedArgumentNotAllowed - src/Validator/AllowTransition/AssertAllowTransitionsValidator.php:32:33 - Method array_merge called with named unpacked array array<array-key, mixed> (array with string keys) (see https://psalm.dev/268)
        $allTo = array_merge(...$allTo);
@BacLuc
api/psalm.xml: fix warnings about future default changes
268452c 
Warning: "findUnusedBaselineEntry" will be defaulted to "true" in Psalm 6. You should explicitly enable or disable this setting.
-> set this to true because we have no errors with it.

Warning: "findUnusedCode" will be defaulted to "true" in Psalm 6. You should explicitly enable or disable this setting.
-> set this to false because with the injection framework, we have a lot of "unused" constructors.
@BacLuc
api/contentNode: add required psalm template parameters
db97f1e 
This fixes the following psalm errors:
ERROR: MissingTemplateParam - src/Repository/ContentNodeRepository.php:19:7 - App\Repository\ContentNodeRepository has missing template params when extending App\Repository\SortableServiceEntityRepository, expecting 1 (see https://psalm.dev/182)
class ContentNodeRepository extends SortableServiceEntityRepository implements CanFilterByUserInterface {

ERROR: MissingTemplateParam - src/Repository/ColumnLayoutRepository.php:14:7 - App\Repository\ColumnLayoutRepository has missing template params when extending App\Repository\ContentNodeRepository, expecting 1 (see https://psalm.dev/182)
class ColumnLayoutRepository extends ContentNodeRepository {

ERROR: MissingTemplateParam - src/Repository/MaterialNodeRepository.php:14:7 - App\Repository\MaterialNodeRepository has missing template params when extending App\Repository\ContentNodeRepository, expecting 1 (see https://psalm.dev/182)
class MaterialNodeRepository extends ContentNodeRepository {

ERROR: MissingTemplateParam - src/Repository/MultiSelectRepository.php:14:7 - App\Repository\MultiSelectRepository has missing template params when extending App\Repository\ContentNodeRepository, expecting 1 (see https://psalm.dev/182)
class MultiSelectRepository extends ContentNodeRepository {

ERROR: MissingTemplateParam - src/Repository/SingleTextRepository.php:14:7 - App\Repository\SingleTextRepository has missing template params when extending App\Repository\ContentNodeRepository, expecting 1 (see https://psalm.dev/182)
class SingleTextRepository extends ContentNodeRepository {

ERROR: MissingTemplateParam - src/Repository/StoryboardRepository.php:14:7 - App\Repository\StoryboardRepository has missing template params when extending App\Repository\ContentNodeRepository, expecting 1 (see https://psalm.dev/182)
class StoryboardRepository extends ContentNodeRepository {
@usu usu merged commit a4266f8 into devel Feb 4, 2023
@carlobeltrame carlobeltrame deleted the renovate/major-php-linter branch February 4, 2023 18:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@BacLuc BacLuc BacLuc approved these changes

@usu usu usu approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@BacLuc @usu