You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Codecov has the notion of upload limitation for certain plans, historically only users-basic, for a flat value of 250 rolling upload limitations. With the introduction of the team plan, we want to ensure we make a dynamic selection of monthly uploads to applicable plans.
This change involves mostly adjusting logic in this file https://github.com/codecov/worker/blob/main/services/decoration.py#L157 and revolving logic to determine upload limitation taking into account the plan name. There's not a plan representation object equivalent to the one we have in API, so I'd suggest creating some basic dictionary with plans that can have upload limitations and they're respective values, something like
And using some logic along the lines of if plan in plans_with_upload_limitations then upload_limitations = plans_with_upload_limitations[plan_name] and ensure tests pass
The text was updated successfully, but these errors were encountered:
Codecov has the notion of upload limitation for certain plans, historically only
users-basic
, for a flat value of 250 rolling upload limitations. With the introduction of the team plan, we want to ensure we make a dynamic selection of monthly uploads to applicable plans.This change involves mostly adjusting logic in this file https://github.com/codecov/worker/blob/main/services/decoration.py#L157 and revolving logic to determine upload limitation taking into account the plan name. There's not a plan representation object equivalent to the one we have in API, so I'd suggest creating some basic dictionary with plans that can have upload limitations and they're respective values, something like
And using some logic along the lines of
if plan in plans_with_upload_limitations then upload_limitations = plans_with_upload_limitations[plan_name]
and ensure tests passThe text was updated successfully, but these errors were encountered: