Skip to content
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

Added url to python implementation #16

Closed
wants to merge 1 commit into from
Closed

Added url to python implementation #16

wants to merge 1 commit into from

Conversation

djetelina
Copy link

@djetelina djetelina commented Nov 13, 2018

Hopefully this isn't too shameless :) - Once it's more idiomatic, I'll gladly pull request the implementation standalone into here.

Hopefully this isn't too shameless :) - Once it's more idiomatic, I'll gladly push the implementations standalone into here.
@djetelina
Copy link
Author

Any thoughts on this? Perhaps a suggestion for modifycation of the PR?

@nunesvictor
Copy link

Maybe you can fork the repo and add your implementation to it... I think it will be better to keep all the implementations is the same place.

@nunesvictor
Copy link

I was gonna say I was about to do so, but I saw your PR and think maybe you have some priority on it.

@djetelina
Copy link
Author

djetelina commented Nov 20, 2018

@nunesvictor To be frank, I'd love for my library to be the recommended wait of using this and all future APIs in Python.
In my mind it makes more sense to have an opensource updated library that everyone uses than an implementation without tests and means of distribution in this repository that's copied to every project.
But - I know it's unlikely for Valve to endorse my library without at least establishing trust first - so I don't mind adjusting the implementation and posting it in full into a python directory in this repo - but I'd love to hear from somebody at Valve first, seeing as #10 isn't getting merged either.

@djetelina djetelina closed this by deleting the head repository Oct 23, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants