-
Notifications
You must be signed in to change notification settings - Fork 0
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
How do we handle forks of 3rd party repositories #18
Comments
We're not going to guess. We'll just open an issue to ask the maintainer(s) if they want a new feature we're considering.
If it's because of a technical issue, we fix the technical issue. Because we ask about features beforehand, we don't put ourselves in this position for non-technical issues.
Again, we're not going to put ourselves in a position where we don't know if a feature will be accepted by a repo's maintainer(s). So from the perspective of rejected pull requests, this is a non-starter. We fork for two reasons:
Determining if we fork and -- if we do -- which direction we take the fork is entirely contextual. There's never a right/wrong choice.
We can always roll our own. That's something we'll consider, but just like forking and maintaining, it's entirely contextual. |
Thanks for the feedback @jdenen very helpful! When it comes to releasing a thing would we hold that up until the upstream issue / pr is merged/resolved or point at our fork? |
It depends. If it's something that we can build/bundle/deploy easily, we'd point at our fork until the fix or feature is merged. Then, we'd do a small deployment to point at the updated repo when that happens. If it's something that would require us to publish a forked artifact to use (think a Ruby gem in rubygems.org, etc), we would wait. This is still pretty highly dependent on the context. There are often temporary workarounds for these situations, but it's not something we can really plan ahead for. |
This may relate to milestone 2 and it may also be better suited to milestone 3.
A discussion was started within the slack channel.
There are a few gotchas related to this question.
This list is not intended to be exhaustive but something to capture a few of the concerns
The text was updated successfully, but these errors were encountered: