Skip to content

prevent the installation of unstable phpdocumentor/type-resolver releases #48471

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

Merged
merged 1 commit into from
Dec 4, 2022

Conversation

xabbuh
Copy link
Member

@xabbuh xabbuh commented Dec 4, 2022

Q A
Branch? 5.4
Bug fix? no
New feature? no
Deprecations? no
Tickets
License MIT
Doc PR

I suggest to not make use of the development version of the upcoming minor release of phpdocumentor/type-resolver as it is quite unstable at the moment and causes our pipeline to fail constantly.

@xabbuh xabbuh added this to the 5.4 milestone Dec 4, 2022
@xabbuh xabbuh requested a review from dunglas as a code owner December 4, 2022 17:23
Copy link
Member

@chalasr chalasr left a comment

Choose a reason for hiding this comment

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

🎉

@nicolas-grekas
Copy link
Member

I'm mixed on this. We do want to spot regressions before they are tagged. I'm fine merging but only if we revert very soon.

@xabbuh xabbuh merged commit 7e30473 into symfony:5.4 Dec 4, 2022
@xabbuh xabbuh deleted the pr-48332 branch December 4, 2022 18:22
@xabbuh
Copy link
Member Author

xabbuh commented Dec 4, 2022

@nicolas-grekas I didn't spot your comment before merging. I will closely monitor the phpdocumentor/type-resolver repository to revert this change as soon as possible.

@Gemorroj
Copy link
Contributor

xabbuh added a commit that referenced this pull request Mar 19, 2023
…abbuh)

This PR was merged into the 5.4 branch.

Discussion
----------

[Serializer] re-allow phpdocumentor/type-resolver 1.7

| Q             | A
| ------------- | ---
| Branch?       | 5.4
| Bug fix?      | no
| New feature?  | no
| Deprecations? | no
| Tickets       | reverts #48471
| License       | MIT
| Doc PR        |

Commits
-------

0776dec re-allow phpdocumentor/type-resolver 1.7
@xabbuh
Copy link
Member Author

xabbuh commented Mar 19, 2023

reverted in #49727

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants
pFad - Phonifier reborn

Pfad - The Proxy pFad of © 2024 Garber Painting. All rights reserved.

Note: This service is not intended for secure transactions such as banking, social media, email, or purchasing. Use at your own risk. We assume no liability whatsoever for broken pages.


Alternative Proxies:

Alternative Proxy

pFad Proxy

pFad v3 Proxy

pFad v4 Proxy