search

PyWavelets / pywt

PyWavelets - Wavelet Transforms in Python
http://pywavelets.readthedocs.org
MIT License
2.04k stars 470 forks source link

update Tidelift link in the README [ci skip] #532

Closed grlee77 closed 4 years ago

grlee77 commented 4 years ago

This PR updates the Tidelift link in the README to take advantage of a promotion they are currently running. Basically, they will pay the project a bonus for any sales leads that result from users clicking on the link in this form.

Tidelift also has provided a template for adding a "For Enterprise" page to the documentation that provides more context and similar links/buttons. I chose not to do that as part of this PR, but if others are in favor we can add it so users browsing on readthedocs would find a link to Tidelift as well. Adding the documentation page is optional and is not a requirement for continuing Tidelift support.

rgommers commented 4 years ago

I'm actually in favor of a "for enterprise" or similar page, which explains things like how a company should engage with us as an open source project if they want support, want to add major features, etc. I think this is something all community driven projects that are at or near the center of the SciPy/PyData ecosystem should have.

I do object to some of the Tidelift language though - they don't actually provide everything they claim (in particular, the claim about support is far too broad).

rgommers commented 4 years ago

This link tweak seems useful and uncontroversial, so merging

phated commented 4 years ago

Hey @grlee77 and @rgommers, I noticed you changed the UTM parameter but you didn't add the enterprise language. To pay for workable leads, we need you to implement the enterprise copy as well. Is there something I can do to help?

rgommers commented 4 years ago

Hi @phated, thanks for pointing that out. Reading the email discussion with Brenna, it indeed looks to me like we should have done all changes or nothing. @grlee77 am I missing something? Should we revert for now?

@phated happy to chat about the enterprise support language. Email is better perhaps?

grlee77 commented 4 years ago

Yeah, I think I had misunderstood. We can just revert this PR for now and add the link in with some (possibly modified) enterprise language in a future PR.