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

Validate buildpack API version #282

Merged
merged 3 commits into from
Sep 9, 2019
Merged

Conversation

jromero
Copy link
Member

@jromero jromero commented Sep 5, 2019

Also:

  • Default lifecycle during create-builder if no version or URI is specified
  • Use BP API Version to determine order.toml format

Resolves #254
Resolves #284

Signed-off-by: Javier Romero [email protected]
Signed-off-by: Andrew Meyer [email protected]

@jromero jromero force-pushed the feature/260-lifecycle-descriptor branch from d92e1c6 to ef9b4f2 Compare September 6, 2019 01:52
@jromero jromero force-pushed the feature/254-validate-buildpack-api branch from 4dda93e to 06359fa Compare September 6, 2019 15:02
@jromero jromero changed the base branch from feature/260-lifecycle-descriptor to master September 6, 2019 15:03
@jromero jromero force-pushed the feature/254-validate-buildpack-api branch from 06359fa to 85c11e8 Compare September 6, 2019 15:06
@jromero jromero requested review from djoyahoy and ekcasey and removed request for djoyahoy September 6, 2019 15:48
api/version.go Outdated Show resolved Hide resolved
api/version.go Outdated Show resolved Hide resolved
api/version_test.go Outdated Show resolved Hide resolved
api/version_test.go Outdated Show resolved Hide resolved
builder/builder.go Outdated Show resolved Hide resolved
builder/lifecycle.go Outdated Show resolved Hide resolved
build/build.go Outdated Show resolved Hide resolved
builder/builder_test.go Outdated Show resolved Hide resolved
build_test.go Outdated Show resolved Hide resolved
create_builder_test.go Outdated Show resolved Hide resolved
* Default lifecycle during `create-builder` if no version or URI is specified

Resolves #254

Signed-off-by: Javier Romero <[email protected]>
Signed-off-by: Andrew Meyer <[email protected]>
@jromero jromero force-pushed the feature/254-validate-buildpack-api branch 2 times, most recently from bac0a66 to d0dcd17 Compare September 9, 2019 21:07
@jromero jromero requested a review from ekcasey September 9, 2019 21:07
@jromero jromero force-pushed the feature/254-validate-buildpack-api branch 2 times, most recently from d915cb0 to 016922c Compare September 9, 2019 21:16
* Fix implementation of `api.Version#SupportsVersion`
* Remove AdditionalBuildpack wrapper

Signed-off-by: Javier Romero <[email protected]>
Signed-off-by: Andrew Meyer <[email protected]>
@jromero jromero force-pushed the feature/254-validate-buildpack-api branch from 016922c to 34efdee Compare September 9, 2019 21:48
@jromero jromero merged commit dd15e5c into master Sep 9, 2019
@ekcasey ekcasey deleted the feature/254-validate-buildpack-api branch October 29, 2019 19:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Use the lifecycle's Buildpack API Version to determine order.toml format Validate buildpack API version
2 participants