Skip to content

Conversation

@antograssiot
Copy link
Contributor

@antograssiot antograssiot commented Oct 13, 2020

Q A
Bug fix? no
New feature? no
BC breaks? no
Deprecations? no
Tickets
License MIT
Doc PR

We might want to skip the installation of symfony/uid for legacy tests completely. The rationale behind this is that we can't install this package when the symfony requirement is limited to ^3.4 | ^4.0 only.

@dunglas
Copy link
Member

dunglas commented Oct 14, 2020

But then we don't test the use of this component at all? Maybe should we add a new job just for that?

@soyuka
Copy link
Member

soyuka commented Oct 14, 2020

see also #3715

We do test the component 👍 to not install this on legacy versions

@antograssiot
Copy link
Contributor Author

Yes we do test it in other pipelines but on on the legacy suite that runs only on SF ^3.4|^4.0

@soyuka soyuka merged commit 87c75d4 into api-platform:master Oct 14, 2020
@soyuka
Copy link
Member

soyuka commented Oct 14, 2020

thansk @antograssiot

@antograssiot antograssiot deleted the fix-pipeline branch October 14, 2020 09:38
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.

3 participants