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

Add Tidelift management docs and blurb on README, as discussed in the ML #5240

Merged
merged 1 commit into from
May 10, 2019

Conversation

nicoddemus
Copy link
Member

As discussed in the ML, adding a document describing how we plan to split the funds, and how maintainers can start/stop receiving funding.

A quick draft, so suggestions on the docs are very welcome!

@asottile and @blueyed can already start their subscription process. Payments occur at the end of each month.

@RonnyPfannschmidt would you also like to join right now?


I created @pytest-dev/tidelift-admins to list the people which can make changes to Tidelift. What do you folks think?

cc @kszu

Copy link
Member

@asottile asottile left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks great, thanks for writing this up and doing the due diligence about documenting the admin process 👍

TIDELIFT.rst Show resolved Hide resolved
@codecov
Copy link

codecov bot commented May 9, 2019

Codecov Report

Merging #5240 into master will increase coverage by <.01%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #5240      +/-   ##
==========================================
+ Coverage   96.06%   96.06%   +<.01%     
==========================================
  Files         114      114              
  Lines       25823    25823              
  Branches     2550     2550              
==========================================
+ Hits        24806    24808       +2     
+ Misses        706      705       -1     
+ Partials      311      310       -1
Impacted Files Coverage Δ
testing/acceptance_test.py 98.03% <0%> (+0.39%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 803d688...dae455e. Read the comment docs.

Copy link
Contributor

@blueyed blueyed left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you!

TIDELIFT.rst Outdated Show resolved Hide resolved
TIDELIFT.rst Show resolved Hide resolved
@RonnyPfannschmidt
Copy link
Member

@nicoddemus thanks for preparing all this, i will add myself to the admins once i can commit to that role

@nicoddemus
Copy link
Member Author

@RonnyPfannschmidt thanks, but I meant if you would like to also receive funds right now. In my opinion you should, even if you are in a hiatus right now.

@RonnyPfannschmidt
Copy link
Member

Thanks @nicoddemus , i appreciate the offer/sentiment!

But i believe that we should have a very clear cut and pedantic line for the fairness that people who pay those funds will perceive.

right now my position/role is one that shouldn't be paid, so i don't get pay ^^

it is critical to consider, as those that provide the money do not share the nostalgia and gratefulness we feel for each other,
so from their perception something we do in good faith may look like misuse

i want to take part in ensuring that the overall reality is that pytest is grateful for the funds and makes good and effective use of them

naturally resulting from that mindset is that right now i don't get a dime :)

@flub
Copy link
Member

flub commented May 10, 2019

the tidelift-admins team also seems like a good call +1 to all this

@flub flub merged commit 4ccaa98 into pytest-dev:master May 10, 2019
@nicoddemus nicoddemus deleted the tidelift branch May 10, 2019 11:44
@nicoddemus
Copy link
Member Author

Great, thanks everyone!

@asottile @blueyed, make sure to subscribe to Tidelift so you can get your share by the end of the month! 😁

cc @pytest-dev/core

@blueyed
Copy link
Contributor

blueyed commented May 20, 2019

btw: just found this (a script to upload release notes): nedbat/coveragepy@e6bd510

@pfctdayelise
Copy link
Contributor

Are these the only two docs pages not linked in Contents, somehow?

AKA how tf do I find this Tidelift page on the website?!

@nicoddemus
Copy link
Member Author

@pfctdayelise indeed my bad, forgot to add a link in the contents page. :/

nicoddemus added a commit to nicoddemus/pytest that referenced this pull request May 21, 2019
@nicoddemus
Copy link
Member Author

PR open: #5297

Sup3rGeo pushed a commit to Sup3rGeo/pytest that referenced this pull request May 30, 2019
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.

6 participants