PR#8: Update .zuul.yaml - fedora-infra/toddlers

fedora-infra / toddlers

#8 Update .zuul.yaml

Merged 3 years ago by nphilipp. Opened 3 years ago by fbo.

fedora-infra/ fbo/toddlers zuul into master

Update .zuul.yaml

Fabien Boucher • 3 years ago

3edc185

.zuul.yaml

file modified

+3 -3

		`@@ -1,5 +1,5 @@`
		`---`
		`- project:`
		`- check:`
		`- jobs:`
		`- - tox`
		`+ check:`
		`+ jobs:`
		`+ - tox`

no initial comment

pingou commented 3 years ago

Ahah pesky indentation, thanks for the fix :)

pingou commented 3 years ago

btw, to run the tests locally we need to call tox ./tests, calling just tox will not work. Do we need to do the same here? (it looks like the job isn't managing to start on zuul atm)

nphilipp commented 3 years ago

Here, just running tox from the root of the source tree works.

zuul commented 3 years ago

Build failed.

tox : FAILURE in 2m 04s

nphilipp commented 3 years ago

Zuul attempts to run the tox job, so the PR is good. As discussed on IRC, I'll work on getting Zuul's tox job(s) set up properly in another PR. Merging.