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

Update spec to specify expected version format #85

Conversation

danielleadams
Copy link
Member

@danielleadams danielleadams commented Jun 3, 2020

Updates spec following RFC #35 that will add version format guidance to buildpack authors: https://github.com/buildpacks/rfcs/blob/master/text/0035-buildpack-versions.md

Addresses: #72

jkutner and others added 15 commits January 22, 2020 20:37
Signed-off-by: Joe Kutner <jpkutner@gmail.com>
Signed-off-by: Joe Kutner <jpkutner@gmail.com>
Signed-off-by: Joe Kutner <jpkutner@gmail.com>
* the source of these binding will be platform specific

Signed-off-by: Emily Casey <ecasey@pivotal.io>
The platform should be responsible for injecting the bindings
Signed-off-by: Joe Kutner <jpkutner@gmail.com>
Signed-off-by: Joe Kutner <jpkutner@gmail.com>
Signed-off-by: Joe Kutner <jpkutner@gmail.com>
Signed-off-by: Joe Kutner <jpkutner@gmail.com>
Previously, the specification required a "tags" file that contained a set of
arbitrary data.  This hasn't been used for anything particularly useful to
date, and it has been pointed out that allowing arbitrary files in metadata
can achieve the same ends.

Because of these observations, this change removes tags from the
specification.

Signed-off-by: Ben Hale <bhale@pivotal.io>
Signed-off-by: Ben Hale <bhale@vmware.com>
Signed-off-by: Ben Hale <bhale@vmware.com>
Signed-off-by: Ben Hale <bhale@vmware.com>
@danielleadams danielleadams force-pushed the update-buildpack-version-specs branch from 44eda24 to 3148b5d Compare June 3, 2020 18:49
buildpack.md Outdated Show resolved Hide resolved
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 just added some wording updates.

@nebhale
Copy link
Contributor

nebhale commented Jun 5, 2020

@danielleadams Can you please do a rebase and force push to clear up the DCO issue? There needs to be a signoff line for every commit.

Co-authored-by: Terence Lee <hone02@gmail.com>
Signed-off-by: Danielle Adams <danielle.adams@heroku.com>
@danielleadams danielleadams force-pushed the update-buildpack-version-specs branch from f235649 to d85f3c0 Compare June 5, 2020 14:57
@nebhale nebhale changed the base branch from master to buildpack/0.3 June 8, 2020 16:07
@nebhale
Copy link
Contributor

nebhale commented Jun 8, 2020

Resolved by 4ddee61

@nebhale nebhale closed this Jun 8, 2020
@ekcasey ekcasey added this to the Buildpack 0.3 milestone Jun 24, 2020
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.

5 participants