gittuf / gittuf

A security layer for Git repositories
https://gittuf.dev
Apache License 2.0
439 stars 28 forks source link

ci: Enable `get-started` tester on certain docs changes #442

Closed patzielinski closed 1 week ago

patzielinski commented 1 week ago

We should run the get-started CI tests if there are changes to the get-started.md doc or testing data.

adityasaky commented 1 week ago

for the build gittuf step, could you also update the Makefile so that we have:

install : test just-install

just-install:
    CGO_ENABLED=0 go install -trimpath -ldflags "$(LDFLAGS)" github.com/gittuf/gittuf

And in the CI job, instead of make, let's run make just-install. I see no reason to run tests twice in CI effectively.

adityasaky commented 1 week ago

See: https://github.com/gittuf/gittuf/actions/runs/9520506484/job/26246067112#step:5:12