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

Add HIP template #2

Merged
merged 1 commit into from
Aug 20, 2019
Merged

Add HIP template #2

merged 1 commit into from
Aug 20, 2019

Conversation

JayKickliter
Copy link
Contributor

@JayKickliter JayKickliter commented Aug 16, 2019

This PR adds a template for HIPs. This template is sourced from @chrisabruce, and I'm assuming he modified it from some other upstream source?

This template is not necessarily in its final form, but a starting point. All future non-administrative PRs to this repo will use this template for their proposals.

@JayKickliter JayKickliter requested review from Vagabond, madninja, chrisabruce and amirhaleem and removed request for Vagabond August 16, 2019 20:23
@JayKickliter
Copy link
Contributor Author

If HIPs are going to be successful, this is the one place where bikeshedding is actually desired and crucial. That said, I'm going to merge this.

@JayKickliter JayKickliter merged commit 3dfe920 into master Aug 20, 2019
@amirhaleem amirhaleem deleted the jsk/add-template branch February 18, 2020 21:41
fvasquez added a commit that referenced this pull request Mar 23, 2020
@fvasquez fvasquez mentioned this pull request Mar 23, 2020
fvasquez pushed a commit that referenced this pull request Mar 23, 2020
@philknows philknows mentioned this pull request May 7, 2021
shayons297 added a commit to shayons297/HIP that referenced this pull request Feb 23, 2022
abhay pushed a commit that referenced this pull request Feb 23, 2022
samgutentag pushed a commit that referenced this pull request Mar 16, 2023
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.

3 participants