Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Exporter: when no default process is specified, don't set a default process (even when there is only one process) #137

Merged

Conversation

natalieparellano
Copy link
Member

@natalieparellano natalieparellano commented Aug 18, 2020

This is in service of buildpacks/lifecycle#378

The lifecycle should be as transparent as possible and have no opinions. The current behavior could be surprising to some users.

Goes hand-in-hand with #159

Signed-off-by: Natalie Arellano narellano@vmware.com

…rocess (even when there is only one process)

The lifecycle should be as transparent as possible and have no opinions. The current behavior could be surprising to some users.

Signed-off-by: Natalie Arellano <narellano@vmware.com>
@natalieparellano natalieparellano requested a review from a team as a code owner August 18, 2020 18:49
@hone
Copy link
Member

hone commented Aug 19, 2020

This is technically a breaking change. I'm not sure if any/many folks would be impacted by it.

@ekcasey ekcasey changed the base branch from main to buildpack/0.5 August 19, 2020 15:47
@ekcasey
Copy link
Member

ekcasey commented Aug 19, 2020

repointed at platform/0.5

@ekcasey ekcasey added this to the Platform 0.5 milestone Aug 19, 2020
Copy link
Member

@ekcasey ekcasey left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I am fine with this change if folks think this provides a better UX.

@ekcasey ekcasey changed the base branch from buildpack/0.5 to platform/0.5 August 19, 2020 18:21
@natalieparellano
Copy link
Member Author

cc @jromero

@natalieparellano
Copy link
Member Author

Related RFC: buildpacks/rfcs#110

Copy link
Member

@hone hone left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm ok approving this, but I'm struggling with how to handle the number of "breaking" changes that are coming through.

@ekcasey
Copy link
Member

ekcasey commented Aug 26, 2020

I am going to put this in draft for now because we would rather accept these changes in combination with those proposed in buildpacks/rfcs#110.

@ekcasey ekcasey marked this pull request as draft August 26, 2020 18:32
@nebhale nebhale requested review from nebhale and a team September 9, 2020 17:43
@nebhale nebhale merged commit adfbbcf into buildpacks:platform/0.6 Dec 2, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants