Skip to content

Conversation

@willemverspyck
Copy link
Contributor

  • Remove checks that are not needed anymore, like method_exists($this->requestStack, 'getSession'), method_exists($this->requestStack, 'getMainRequest') or method_exists($extension, 'addAuthenticatorFactory'), as these methods are available in all supported Symfony versions.
  • Remove deprecated configs
  • Fix PHPStan

Fix missing $

Fix "Received non-retryable error: Failed request: (409) Conflict: an artifact with this name already exists on the workflow run"

Fix PHPStan
@willemverspyck
Copy link
Contributor Author

willemverspyck commented Jan 12, 2026

I tried to repair the PHP Stan errors with small changes and by removing code that was needed for Symfony 5, that is not supported anymore. I still have to do more tests with my changes, but if others can also test my branche, please do 👍

Then we can go forward and allow Symfony 8.

@willemverspyck willemverspyck marked this pull request as ready for review January 12, 2026 12:49
@XWB
Copy link
Member

XWB commented Jan 12, 2026

Seems good to me.

@willemverspyck
Copy link
Contributor Author

I did more tests and this works for me. Please merge @XWB 😊

@XWB XWB merged commit cab845d into hwi:master Jan 14, 2026
8 checks passed
@XWB
Copy link
Member

XWB commented Jan 14, 2026

👍

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