-
-
Notifications
You must be signed in to change notification settings - Fork 450
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
[Proposal] Feature Discussion From Our Fork #374
Comments
Thanks for the proposal, I think many of these features are well seasoned and will be warmly welcomed by users! So definitively, it would be great if all these improvements can be merged into I personally don't have much time to do OSS lately, but I'm happy to ship these features in a future Would you or your organization be interested in becoming contributors? I can invite you, otherwise, PRs will work too. |
I'll let you know! @h2non thanks for responding, I'd definitely be interested in helping out on my personal time, need to make sure that's okay when working with org code. We did some major reworks to the API which is why we'd probably have to make this changes added piece meal and then update them. One example of this is we changed the middleware from |
Any update on this please @angelo-hub?
I've gathered some useful PRs and merged them together, fixing all the merge issues, and the end result is: https://github.com/suntong/imaginary/commits/dev (the commit logs had the merged PRs before, but they're all gone after your taking #353 and me doing a rebase on that). Take a look and let me know if you want me to send in a PR. Thanks |
Currently we have an internal fork we're discussing making public, and would like to contribute back to the project.
The following changes have been added:
HEAD
method request to make a proper sized Go buffermockHost
URLpackage main
structure [sources
package,image
package,logger
package,server
package, andentities
package]EDIT:
If any of these changes would interest you guys as maintainers I'm sure they can be brought in as piece meal.
The text was updated successfully, but these errors were encountered: