-
Notifications
You must be signed in to change notification settings - Fork 807
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
Regarding Pull requests #4
Comments
I believe that this approach is not so convenient and friendly to contributors 🤔 |
It makes sense to me that engaged contributors would like to work differently but I cannot easily change our internal development process. Note that between our 4.0 github release in December and today we have received 9 github pull requests which were as far as I can remember all good changes, and a big thank you goes out to the respective contributors. We have pulled them into mainline and will be pushing them back out in the next update soon. However in the same time our internal dev branch has seen well over 200 checkins. So its safe to say that by far the majority of work still happens internally and we need to keep that process as streamlined as possible. Also I am not sure if everyone would appreciate that volume of daily churn. I think periodic tested releases are more convenient for many people. |
PhysX does not come with a gitignore because the PhysX Git repo is actually a mirror of a Perforce repo. (NVIDIAGameWorks#4)
Hi there,
our legal counsel has advised us to put a CLA in place before accepting pull requests. We are working on sorting this out but due to the impending holidays it won't happen until next year.
Of course if you find and fix issues there is no harm in queuing up the fixes already so they are not forgotten about.
Another thing to be aware of is that our internal day to day development happens in perforce and we will continue pushing periodic 'releases' over to github that have passed our internal testing process. So even when we start to take pull requests they will not manifest on github immediately, but will get merged to perforce first and then appear here later as part of the next versioned update.
--Adam
The text was updated successfully, but these errors were encountered: