Skip to content

Try to fix master pipeline - #3756

Merged
soyuka merged 1 commit into
api-platform:masterfrom
antograssiot:fix-pipeline
Oct 14, 2020
Merged

Try to fix master pipeline#3756
soyuka merged 1 commit into
api-platform:masterfrom
antograssiot:fix-pipeline

Conversation

@antograssiot

@antograssiot antograssiot commented Oct 13, 2020

Copy link
Copy Markdown
Contributor
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

dunglas commented Oct 14, 2020

Copy link
Copy Markdown
Member

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

@soyuka

soyuka commented Oct 14, 2020

Copy link
Copy Markdown
Member

see also #3715

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

@antograssiot

Copy link
Copy Markdown
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

soyuka commented Oct 14, 2020

Copy link
Copy Markdown
Member

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