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

RFC issue generation #141

Closed
wants to merge 7 commits into from
Closed

RFC issue generation #141

wants to merge 7 commits into from

Conversation

jromero
Copy link
Member

@jromero jromero commented Mar 3, 2021

Signed-off-by: Javier Romero <rjavier@vmware.com>
@hone
Copy link
Member

hone commented Mar 3, 2021

🙏

Signed-off-by: Javier Romero <rjavier@vmware.com>
Signed-off-by: Javier Romero <rjavier@vmware.com>
Signed-off-by: Javier Romero <rjavier@vmware.com>
@jromero jromero marked this pull request as ready for review March 5, 2021 15:10
@jromero jromero requested a review from a team as a code owner March 5, 2021 15:10
@nebhale nebhale requested a review from a team March 17, 2021 19:15
@hone
Copy link
Member

hone commented Mar 17, 2021

I really like the comments idea. What are people's thoughts on making that the proposal vs. the alternative?


##### Cons

- No call-to-action for maintainers. The maintainers would have to remember to do this during their review.
Copy link

Choose a reason for hiding this comment

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

Perhaps the RFC merge script could fail if it detects a total of 0 net issues created.

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'm not sure this would be a healthy check to impose. Looking through some RFCs, in particular those related to process or governance changes, it's unclear that any issues should be created. If we think hard enough, if only to satisfy the requirement, I'm sure we can find some issue to create but not sure it's the best approach.

Some examples:

@jromero
Copy link
Member Author

jromero commented Mar 24, 2021

I really like the comments idea. What are people's thoughts on making that the proposal vs. the alternative?

I'm going to make that the primary proposal.

Signed-off-by: Javier Romero <rjavier@vmware.com>
Signed-off-by: Javier Romero <rjavier@vmware.com>
@jromero
Copy link
Member Author

jromero commented Apr 2, 2021

I really like the comments idea. What are people's thoughts on making that the proposal vs. the alternative?

I'm going to make that the primary proposal.

This has been done. @buildpacks/core-team please re-review.

> _(none)_
>

Once the RFC is merged via the [`merge-rfc.sh`](https://github.com/buildpacks/rfcs/blob/main/merge-rfc.sh) script, the script could detect issues by looking through the top comment from the _bot_ and creating any queued issues. Some additions that the script may do but subject to change:
Copy link
Member

Choose a reason for hiding this comment

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

I assume this could be done via a GitHub action on PR merges?

Copy link
Member Author

Choose a reason for hiding this comment

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

At a later point in time or via separate initiative, yes.

@nebhale nebhale requested a review from a team April 7, 2021 19:11
@nebhale nebhale requested a review from a team April 7, 2021 19:11
@sclevine
Copy link
Member

sclevine commented Apr 7, 2021

FCP closing 4/14/21

Signed-off-by: Javier Romero <rjavier@vmware.com>

Co-authored-by: Anthony Emengo <anthonyemengojr@gmail.com>
@sclevine
Copy link
Member

@jromero to help with issue creation

@hone
Copy link
Member

hone commented May 5, 2021

Merged this RFC.

@hone hone closed this May 5, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

8 participants