-
Notifications
You must be signed in to change notification settings - Fork 71
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 for pack detect #308
RFC for pack detect #308
Conversation
Maintainers, As you review this RFC please queue up issues to be created using the following commands:
Issues(none) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for kicking this off @rashadism! I would love to see this happen :)
Thank you @natalieparellano , will update on the stuff you pointed out. |
I opened a corresponding draft PR . @natalieparellano can u have a look |
@jjbustamante is going to steward this one :) |
nice, looking forward to it (: |
I think this one might be ready for another round of reviews. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This looks good to me. I think we need to fix DCO :)
Maintainers, please review! I'll move this one to "voting" on Thursday. |
Signed-off-by: Rashad Sirajudeen <[email protected]>
Co-authored-by: Daniel Mikusa <[email protected]> Signed-off-by: Rashad Sirajudeen <[email protected]>
Signed-off-by: Rashad Sirajudeen <[email protected]>
Signed-off-by: Rashad Sirajudeen <[email protected]>
@rashadism can I get your acceptance on the DCO? I can mark it passed and merge if you'll reply back with a 👍 |
A basic implementation on
pack detect
.Looking forward on getting feedback and improving on this and coming up with the best version of this, and implenenting this.