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.

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

Allow Symfony 6 #33

Merged
merged 4 commits into from
Nov 3, 2021
Merged

Allow Symfony 6 #33

merged 4 commits into from
Nov 3, 2021

Conversation

kbond
Copy link
Contributor

@kbond kbond commented Nov 2, 2021

No description provided.

@kbond kbond marked this pull request as draft November 2, 2021 12:52
@kbond kbond force-pushed the sf6 branch 2 times, most recently from da05a0e to 6b1bde9 Compare November 2, 2021 13:03
@kbond
Copy link
Contributor Author

kbond commented Nov 2, 2021

phpspec/prophecy#527 is the remaining blocker. My suggestion would be another job outside the matrix that runs just phpspec on 7.4. WDYT?

@kbond kbond marked this pull request as ready for review November 2, 2021 13:24
Copy link
Owner

@umpirsky umpirsky left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@umpirsky
Copy link
Owner

umpirsky commented Nov 2, 2021

If you don't have time to wait for phpspec/prophecy#527 to be resolved, we can do that and create issue to update once phpspec/prophecy#527 is fixed.

@kbond
Copy link
Contributor Author

kbond commented Nov 2, 2021

Is it important to run phpspec on every matrix environment? I'm not familiar with this library.

We sort of have 2 options:

  1. Move the phpspec test to it's own job running in a compatible environment
  2. Disable phpspec for this PR, when merged, I'll open another PR to re-add where we can wait on PHP 8 | 'static' is an invalid class name phpspec/prophecy#527

@umpirsky
Copy link
Owner

umpirsky commented Nov 2, 2021

We can do 2. It's desirable to run it on every matrix if possible. Thanks.

@kbond
Copy link
Contributor Author

kbond commented Nov 2, 2021

Ok, this is ready. I disabled phpspec on Symfony 6 only.

@umpirsky umpirsky merged commit e1e3e8d into umpirsky:master Nov 3, 2021
@kbond kbond deleted the sf6 branch November 3, 2021 12:00
@kbond
Copy link
Contributor Author

kbond commented Jan 12, 2022

@umpirsky, could I trouble you for a new tag?

@umpirsky
Copy link
Owner

Done https://github.com/umpirsky/UmpirskyI18nRoutingBundle/releases/tag/0.0.6

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