Skip to content
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

sudo in matrix.include not recognized as correct. #66

Open
bilderbuchi opened this issue Jul 24, 2015 · 7 comments
Open

sudo in matrix.include not recognized as correct. #66

bilderbuchi opened this issue Jul 24, 2015 · 7 comments

Comments

@bilderbuchi
Copy link

When putting a snippet like

sudo: false
env:
  matrix:
    CI_TARGET=A
matrix:
  include:
    - env: CI_TARGET=B
      sudo: true

I get "in matrix.include section: unexpected key sudo, dropping", but this works on Travis.

@cirocosta
Copy link

cirocosta commented Feb 25, 2017

can't reproduce; here this works:

sudo: false
language: go

go:
  - master

env:
  - TEST_SUITE="unit"

matrix:
  include:
    - env: "TEST_SUITE=integration"
      sudo: true

@bilderbuchi
Copy link
Author

@LebedevRI do you agree? I don't know, as @cirocosta's repro is not identical to mine. I don't use Travis right now.

@LebedevRI
Copy link

@cirocosta we are talking about https://lint.travis-ci.org/, right?
Your snippet results in the exact same message: in matrix.include section: unexpected key sudo, dropping

@cirocosta
Copy link

ooh sorry, that's true. I was thinking about travis in general. As you said, fails on lint but works fine on travis. My bad

@PeterDaveHello
Copy link

So this one could be closed maybe?

@LebedevRI
Copy link

Why? It is still reproducible.

I'd guess it should be reported to https://github.com/travis-ci/travis-lint, but the issues are closed there. And i can't rename this issue to contain [lint] prefix.

@PeterDaveHello
Copy link

Oh, sorry, misunderstood :)

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

No branches or pull requests

4 participants