Skip to content

Allow later versions of nette/utils#1322

Closed
sanmai wants to merge 1 commit intorectorphp:masterfrom
sanmai:patch-1
Closed

Allow later versions of nette/utils#1322
sanmai wants to merge 1 commit intorectorphp:masterfrom
sanmai:patch-1

Conversation

@sanmai
Copy link
Copy Markdown
Contributor

@sanmai sanmai commented Apr 16, 2019

This should be a no brainer. Let's see if CI tests pass.

@sanmai
Copy link
Copy Markdown
Contributor Author

sanmai commented Apr 16, 2019

PHPStan sees some trouble, but it's unrelated to nette/utils: builds on master fail with the very same error.

@TomasVotruba
Copy link
Copy Markdown
Member

TomasVotruba commented Apr 16, 2019

Could you fix it anyway?

@sanmai
Copy link
Copy Markdown
Contributor Author

sanmai commented Apr 16, 2019

How about not in this PR? It's kind hard on mobile. Thank you very much.

@TomasVotruba
Copy link
Copy Markdown
Member

Sure, feel free to open new one

@TomasVotruba
Copy link
Copy Markdown
Member

Replaced by #1323

Thank you for the PR though

@sanmai
Copy link
Copy Markdown
Contributor Author

sanmai commented Apr 16, 2019

No problem, but if you could push a release with that fix... it’ll be great!

@TomasVotruba
Copy link
Copy Markdown
Member

TomasVotruba commented Apr 16, 2019

I plan it in a week with bunch of other fixes. In the meantime this is how you make dependencies work:

{
    "nette/utils": "3.0 as 2.5"
}

TomasVotruba added a commit that referenced this pull request Nov 27, 2021
rectorphp/rector-src@3844627 [DX] make all configure() methods accept direct value object configuration (#1322)
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