Build succeeded.
That does not seems to work as expected, perhaps we could remove that part?
To finalize the addition, perhaps we should use such makefile instead:
export FZCI = $(shell pwd)/FZCI.dhall/package.dhall define generate_file » @echo "Rendering $(1)" » @dhall-to-yaml --generated-comment --file $(1).dhall | python3 -mzuulfmt > $(1).yaml endef all: render install-req render: » @dhall hash --file $$FZCI > /dev/null » $(call generate_file,../fedora-zuul-jobs-config/zuul.d/jobs) » $(call generate_file,../fedora-zuul-jobs/zuul.d/jobs) define clone » @test -d ../$(1)/.git || git clone https://pagure.io/$(1).git ../$(1)/ endef install-req: » @type -p dhall-to-yaml > /dev/null || sudo dnf install -y dhall-json » @python3 -mzuulfmt --help > /dev/null || python3 -mpip install --user zuulfmt » $(class clone,fedora-zuul-jobs-config) » $(class clone,fedora-zuul-jobs)
Lets drop that part too.
1 new commit added
Update REAME.md according to comments
Thanks for the review. I've pushed on new commit to address your comments.
oops, this should be clone
clone
let's use python3 -mzuulfmt
python3 -mzuulfmt
let's also install dhall
adress comments
Metadata Update from @fbo: - Pull-request tagged with: gateit
Build succeeded (gate pipeline).
Pull-Request has been merged by zuul