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

Heroku buildpack #40

Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open

Heroku buildpack #40

wants to merge 3 commits into from

Conversation

loganwright
Copy link
Member

initiating the heroku buildpack pr for comments and discussion

* Proposal: SSWG-0012
* Authors: [KyleF](https://github.com/kylef), [VZSG](https://github.com/vzsg)
* Sponsor(s): TBD
* Review Manager: [Logan Wright](https://github.com/loganwright)
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
* Review Manager: [Logan Wright](https://github.com/loganwright)
* Review Manager: [Tom Doron](https://github.com/tomerd)

* Authors: [KyleF](https://github.com/kylef), [VZSG](https://github.com/vzsg)
* Sponsor(s): TBD
* Review Manager: [Logan Wright](https://github.com/loganwright)
* Status: **Implemented**
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
* Status: **Implemented**
* Status: **Active Review** (June 1 - June 12)


## Detailed design

Already implemennted
Copy link
Contributor

Choose a reason for hiding this comment

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

We need a bit more details here, like what the different stages of the buildpack do (which swift command they run, what configuration options the users have, etc)


* Proposal: SSWG-0012
* Authors: [KyleF](https://github.com/kylef), [VZSG](https://github.com/vzsg)
* Sponsor(s): TBD
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
* Sponsor(s): TBD

@0xTim 0xTim changed the base branch from master to main October 5, 2023 22:39
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.

2 participants