-
Notifications
You must be signed in to change notification settings - Fork 8
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
Downsides of forking 3rd party repositories here #5
Comments
Mhm, wait, we have issues for the pbs-torque repo, which is forked... |
I have to enable them. Now we also have them for slurm. Thanks for making me aware of this! |
Yes, then the question is if you want to have separate development workflows or one. Example: I sent a PR to https://github.com/percyfal/slurm to add more detailed installation instructions. Now you fixed them here, which is great, but you'll have a conflict if the original repo accepts the PR. This is why I thought it would make most sense to ask original maintainers if they agree with moving repos to the Snakemake-Profiles user. |
Yes, we can do that as well, you are right. |
If a repository is forked from another source, its issues will be disabled by default.
If I then report an issue with the profile to the original source, this will not be automatically propagated to this collection.
Wouldn't it be better to move repositories here instead of forking them? (and only forking if they change)
Authors would still get credit (because they are listed as commiters)
(Anyway, this is just a suggestion so feel free to ignore)
The text was updated successfully, but these errors were encountered: