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

Adjust buildplan order #203

Merged
merged 1 commit into from
Oct 20, 2022
Merged

Adjust buildplan order #203

merged 1 commit into from
Oct 20, 2022

Conversation

dmikusa
Copy link
Contributor

@dmikusa dmikusa commented Oct 20, 2022

This PR adjusts the buildplan order such that the more specific buildplans come first and the least specific comes last.

The lifecycle tries buildplans in a specific order and it will stop searching for plans when it hits the first plan where all the provides & requires match up. Having the most specific plans first and least specific plans last ensures that you get a correct match.

See Slack Thread for more details

Signed-off-by: Daniel Mikusa dan@mikusa.com

@dmikusa dmikusa requested a review from a team as a code owner October 20, 2022 04:41
Copy link
Member

@robdimsdale robdimsdale left a comment

Choose a reason for hiding this comment

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

Looks good - thanks for the catch.

@robdimsdale robdimsdale added the semver:minor A change requiring a minor version bump label Oct 20, 2022
This PR adjusts the buildplan order such that the more specific buildplans come first and the least specific comes last.

The lifecycle tries buildplans in a specific order and it will stop searching for plans when it hits the first plan where all the provides & requires match up. Having the most specific plans first and least specific plans last ensures that you get a correct match.

See [Slack Thread for more details](https://paketobuildpacks.slack.com/archives/CUD6UTCJW/p1666240482044119?thread_ts=1665373426.739299&cid=CUD6UTCJW)

Signed-off-by: Daniel Mikusa <dan@mikusa.com>
@robdimsdale robdimsdale enabled auto-merge (rebase) October 20, 2022 13:30
@robdimsdale robdimsdale merged commit 7edf55b into main Oct 20, 2022
@robdimsdale robdimsdale deleted the change-plan-order branch October 20, 2022 13:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
semver:minor A change requiring a minor version bump
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants