Skip to content
Permalink
Browse files
Remove sudo: false from .travis.yml 
Since this repository was not recognized by Travis before 2015-01-01,
there is no need to include `sudo: false` as that is now the default.

- - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - -

From https://docs.travis-ci.com/user/workers/container-based-infrastructure/#Routing-your-build-to-container-based-infrastructure:

 " The default behavior, when no sudo usage is detected in
   any customizable build phases, depends on the date when
   the repository is first recognized by Travis CI:

   * For repos we recognize before 2015-01-01, linux builds
     are sent to our standard infrastructure.

   * For repos we recognize on or after 2015-01-01, linux
     builds are sent to our container-based infrastructure "
  • Loading branch information
@alrra
alrra committed Feb 9, 2016
1 parent 393c87e commit bc3dd58602c6968337b11ebd6a1dde044d43f974
Showing 1 changed file with 0 additions and 1 deletion.
1 .travis.yml
View file
@@ -1,4 +1,3 @@
sudo: false
language: node_js
node_js:
- stable

2 comments on commit bc3dd58

@dolmen
Copy link

@dolmen dolmen commented on bc3dd58 Feb 16, 2016

Choose a reason for hiding this comment

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

I can understand that sudo: false did nothing because it is the default, but I don't see why removing the line is helpful. At least it made explicit in what environment Travis-CI would run the build.
The commit message does not say why the line had to be removed.

@nvartolomei
Copy link

Choose a reason for hiding this comment

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

@dolmen 👏

Please sign in to comment.