skuba icon indicating copy to clipboard operation
skuba copied to clipboard

Accept stdin for `configure`

Open samchungy opened this issue 2 years ago • 4 comments

We rely on skuba configure for our template repo which has specific styling/base configuration for our team. The issue is we cannot easily test that changes to that repository because we cannot test that it is working without running skuba configure. I tried injecting some form values using echo and printf with little success. eg. echo 'some value\n' | skuba configure.

The idea would be to steal a bit of code from skuba init and accept templateData from stdin and use that instead of the form if it is provided.

This thread has some context: https://seekchat.slack.com/archives/CDCPCEPV3/p1652495682762049

samchungy avatar Aug 01 '22 02:08 samchungy

Sorry for tangent, but while you're on the subject: are you also getting bitten by skuba lint being unhappy about skuba.config.js on the template repo itself?

I think that was a rough edge that I forgot we had to polish still 😓

72636c avatar Aug 01 '22 02:08 72636c

We don't currently run anything on the said repo 😅 I was looking to run that and a test deploy after running skuba configure

samchungy avatar Aug 01 '22 03:08 samchungy

To tangent even harder, IIRC it's currently difficult/impossible to exclude files like your own meta pipeline.yml from the clone performed by skuba init. Lots of work to do 🙂

72636c avatar Aug 01 '22 04:08 72636c

To tangent even harder, IIRC it's currently difficult/impossible to exclude files like your own meta pipeline.yml from the clone performed by skuba init. Lots of work to do 🙂

With the test pipeline I plan to create a separate file pipeline-test.yml and get users to just delete that as part of the steps. Then I'll just manually change the BK pipeline Edit Steps to target that for the template repo only

samchungy avatar Aug 01 '22 04:08 samchungy