I feel like I saw this in the docs somewhere but is there a Pants #general

I feel like I saw this in the docs somewhere, but ...

high-yak-85899

02/02/2022, 12:32 AM

I feel like I saw this in the docs somewhere, but is there a way, kind of like a unit test, to make sure that

./pants tailor

doesn't have any changes it would want to apply?

hundreds-father-404

02/02/2022, 12:36 AM

yes,

./pants tailor --check

hundreds-father-404

02/02/2022, 12:36 AM

https://www.pantsbuild.org/docs/create-initial-build-files

high-yak-85899

02/02/2022, 12:37 AM

No, but I mean like I want to invoke that as part of the change process and warn people on commit when it has issues

high-yak-85899

02/02/2022, 12:38 AM

I've been using

--check

heavily already and want to make sure it doesn't blow up with recommended changes as others start getting involved and expanding the scope of what gets built.

hundreds-father-404

02/02/2022, 12:39 AM

I don't think I follow - what do you mean as part of the change process? We recommend adding the specific line

./pants tailor --check

to your CI, but I think I'm not understanding

high-yak-85899

02/02/2022, 12:40 AM

I think that's what I'm getting at. I guess I never inspected the exit code of

tailor

to know how that would be used to stop the build process.

👍 1

high-yak-85899

02/02/2022, 12:41 AM

Looks like it's 0 if there are no changes needed

hundreds-father-404

02/02/2022, 12:46 AM

Yeah, it behaves like

lint

and

test

so that you can depend on it in CI

hundreds-father-404

02/02/2022, 12:46 AM

ditto

update-build-files

busy-vase-39202

02/02/2022, 1:00 AM

@hundreds-father-404 https://www.pantsbuild.org/docs/using-pants-in-ci doesn't mention the exit code. That could be a nice addition, and help clarify for the reader what the value is of the recommended settings.

5 Views

Open in Slack

Previous Next