https://pulumi.com logo
Title
h

helpful-book-29233

11/03/2021, 3:34 AM
Hi! I've been loving pulumi and came across a bug in
pulumi-docker
(#310) I believe I have a fix and wanted to put forward a PR but just wanted to check if I need to name the branch in my fork anything specific (I see other branches using
<username>/<descriptivename>
but can't tell if the username gets applied upon merge or if I need to name the branch that in my fork. (First time contributing to open-source so apologies for the simple question)
b

billowy-army-68599

11/03/2021, 3:36 AM
hey there! any branch name is fine, we run the tests on PR, so just send the PR upstream and ping us and we'll take a look!
thanks so much for contributing!
h

helpful-book-29233

11/03/2021, 4:31 AM
Thanks! Was a really tiny fix (#311). Would really appreciate hearing feedback if I missed anything I should be doing for a PR so I know for future
b

billowy-army-68599

11/03/2021, 4:34 AM
@helpful-book-29233 unfortunately your fix won't quite work, because the SDK folder is automatically generated from the resource itself 😞
h

helpful-book-29233

11/03/2021, 4:38 AM
Ah might have jumped the gun a bit then, any chance you could please point me in the right direction for what controls that?
b

billowy-army-68599

11/03/2021, 4:42 AM
it'll need to be a codegen update, which is out my wheelhouse ability unfortunately...
h

helpful-book-29233

11/03/2021, 5:03 AM
I'm confused about how the codgen works. I saw a previous PR which had edited the same line of code, was that edit generated from this schema?
b

billowy-army-68599

11/03/2021, 5:07 AM
i could be wrong, let's leave the PR open and I'll get some from eng tomorrow to take a look
thanks for diggign in
h

helpful-book-29233

11/03/2021, 5:08 AM
Ok cheers, I've just never worked with codegen before so will be happy to learn more if I've got things mixed up
r

red-match-15116

11/03/2021, 5:23 AM
That's not a codegen file, it's handwritten - so you're all good @helpful-book-29233
👍 1
It's one of the few overlays in the docker provider... most of the rest of it is generated but these files are handwritten: https://github.com/pulumi/pulumi-docker/blob/master/provider/resources.go#L141-L147
❤️ 1