Skip to content

Remove unnecessary historic configuration: 'sudo: false' #1110

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
May 18, 2020
Merged

Remove unnecessary historic configuration: 'sudo: false' #1110

merged 1 commit into from
May 18, 2020

Conversation

michaelKaefer
Copy link
Contributor

I hope I did enough Googling. I really tried to find a reason to keep sudo :false in 2020 but it seems that it has no effect anymore.

I even tried to sudo echo foo during Travis' install phase and that works with and without sudo: false.

Docs: https://docs.travis-ci.com/user/reference/overview/#deprecated-virtualization-environments

Blog post stating "If you currently specify sudo: false in your .travis.yml, we recommend removing that configuration soon": https://blog.travis-ci.com/2018-11-19-required-linux-infrastructure-migration

I also created a PR for the Symfony docs: symfony/symfony-docs#13675

I hope I did enough Googling. I really tried to find a reason to keep `sudo :false` in 2020 but it seems that it has no effect anymore.

I even tried to `sudo echo foo` during Travis' `install` phase and that works with and without `sudo: false`.

Docs: https://docs.travis-ci.com/user/reference/overview/#deprecated-virtualization-environments

Blog post stating "If you currently specify sudo: false in your .travis.yml, we recommend removing that configuration soon": https://blog.travis-ci.com/2018-11-19-required-linux-infrastructure-migration
@javiereguiluz
Copy link
Member

Thanks Michael.

@javiereguiluz javiereguiluz merged commit 7e49b64 into symfony:master May 18, 2020
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