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

Added a project.toml extension #70

Merged
merged 9 commits into from
Apr 24, 2020
Merged

Added a project.toml extension #70

merged 9 commits into from
Apr 24, 2020

Conversation

jkutner
Copy link
Member

@jkutner jkutner commented Jan 23, 2020

Derived from RFC 32.

Signed-off-by: Joe Kutner <jpkutner@gmail.com>
@nebhale nebhale requested review from nebhale, hone and sclevine January 30, 2020 18:16
extensions/project-descriptor.md Outdated Show resolved Hide resolved
extensions/project-descriptor.md Outdated Show resolved Hide resolved
extensions/project-descriptor.md Show resolved Hide resolved
Signed-off-by: Joe Kutner <jpkutner@gmail.com>
Signed-off-by: Joe Kutner <jpkutner@gmail.com>
@jkutner jkutner requested review from nebhale and elbandito February 12, 2020 03:14
@jkutner jkutner marked this pull request as ready for review February 12, 2020 03:15
@jkutner jkutner requested a review from a team as a code owner February 12, 2020 03:15
value = "-Xmx1g"
```

## `[metadata]`
Copy link
Member

@elbandito elbandito Apr 12, 2020

Choose a reason for hiding this comment

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

Hi @jkutner we should remove references to metadata support. While the project.toml file supports metadata, we're currently not doing anything with it. I imagine we'll mostly integrate this functionality with Lifecycle via project-metadata.toml.

Copy link
Member Author

Choose a reason for hiding this comment

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

I think this is meant to be different from project-metadata.toml. This is basically a section for arbitrary metadata about the project/repo, which might not even have anything to do with buildpacks/lifecycle

@sclevine sclevine requested review from ekcasey, hone, sclevine, nebhale, elbandito and jromero and removed request for hone, nebhale and ekcasey April 17, 2020 14:41
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 addition to the schema I think this spec needs to call who consumes it. I.E "a platform SHOULD ..."

@sclevine
Copy link
Member

Apologies, the new GitHub app removed all of the reviewers when I tried to add @ekcasey

Copy link
Member

@jromero jromero left a comment

Choose a reason for hiding this comment

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

Suggested minor edits but content is 👍.

extensions/project-descriptor.md Outdated Show resolved Hide resolved
extensions/project-descriptor.md Outdated Show resolved Hide resolved
jkutner added 3 commits April 18, 2020 07:44
Signed-off-by: Joe Kutner <jpkutner@gmail.com>
Signed-off-by: Joe Kutner <jpkutner@gmail.com>
Signed-off-by: Joe Kutner <jpkutner@gmail.com>
@jkutner
Copy link
Member Author

jkutner commented Apr 18, 2020

@ekcasey @nebhale @jromero feedback addressed

@nebhale nebhale requested a review from a team April 22, 2020 17:08
@jkutner jkutner merged commit 203639f into master Apr 24, 2020
@nebhale nebhale deleted the project-toml branch May 6, 2020 19:30
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.

7 participants