Skip to content

Travis: add new job for scripts/requirements-lax.txt - #237

Merged
deb-intel merged 3 commits into
thesofproject:masterfrom
marc-hb:travis-lax-requirements
Apr 23, 2020
Merged

Travis: add new job for scripts/requirements-lax.txt#237
deb-intel merged 3 commits into
thesofproject:masterfrom
marc-hb:travis-lax-requirements

Conversation

@marc-hb

Copy link
Copy Markdown
Collaborator

See commit messages here and previous, already merged and heavily commented e987dcf for details.

Also upgrade Ubuntu to 18.04 since is two years old now (20.04 is almost released)
This doesn't make much difference anyway since 'language: python' runs everything in a virtualenv.

Pure indentation change to allow the addition of other job(s).
Signed-off-by: Marc Herbert <marc.herbert@intel.com>
18.04 is two years old now and 20.04 is almost released.
This doesn't make much difference anyway since 'language: python' runs
everything in a virtualenv. It will make a difference when we add
another job with another language:
Signed-off-by: Marc Herbert <marc.herbert@intel.com>
LAX=1 so sphinx warnings are OK in this best effort case.
Signed-off-by: Marc Herbert <marc.herbert@intel.com>

@lgirdwoodlgirdwood left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

@intelkevinputnamintelkevinputnam left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM

@deb-intel
deb-intel merged commit b009c66 into thesofproject:masterApr 23, 2020
@marc-hb
marc-hb deleted the travis-lax-requirements branch April 27, 2020 17:28
Sign up for freeto 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.

4 participants

@marc-hb@lgirdwood@intelkevinputnam@deb-intel