feat: count seats for BA notify

[giovanni-guidini]

closes: codecov/engineering-team#2254

Aligning with coverage and test analytics, PR comment for BA will
take up seats now.

[codecov-notifications]

Codecov Report

Attention: Patch coverage is 97.56098% with 2 lines in your changes missing coverage. Please review.

✅ All tests successful. No failed tests found.

Files	Patch %	Lines
services/activation.py	60.00%	2 Missing ⚠️

@@            Coverage Diff             @@
##             main     #654      +/-   ##
==========================================
- Coverage   98.05%   98.05%   -0.01%     
==========================================
  Files         437      437              
  Lines       36849    36881      +32     
==========================================
+ Hits        36134    36165      +31     
- Misses        715      716       +1     

Flag	Coverage Δ
integration	98.05% <97.56%> (-0.01%)	⬇️
latest-uploader-overall	98.05% <97.56%> (-0.01%)	⬇️
unit	98.05% <97.56%> (-0.01%)	⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Components	Coverage Δ
NonTestCode	95.91% <95.55%> (+<0.01%)	⬆️
OutsideTasks	98.06% <97.56%> (-0.01%)	⬇️

Files	Coverage Δ
...ces/bundle_analysis/new_notify/contexts/comment.py	98.75% <100.00%> (+0.33%)	⬆️
.../new_notify/contexts/tests/test_comment_context.py	100.00% <100.00%> (ø)
...ces/bundle_analysis/new_notify/messages/comment.py	100.00% <100.00%> (ø)
...analysis/new_notify/messages/tests/test_comment.py	100.00% <100.00%> (ø)
services/activation.py	93.10% <60.00%> (-6.90%)	⬇️

... and 6 files with indirect coverage changes

[codecov-qa]

Codecov Report

Attention: Patch coverage is 97.56098% with 2 lines in your changes missing coverage. Please review.

Project coverage is 98.05%. Comparing base (aea8367) to head (a607d0a).
Report is 6 commits behind head on main.

✅ All tests successful. No failed tests found.

Files with missing lines	Patch %	Lines
services/activation.py	60.00%	2 Missing ⚠️

@@            Coverage Diff             @@
##             main     #654      +/-   ##
==========================================
- Coverage   98.05%   98.05%   -0.01%     
==========================================
  Files         437      437              
  Lines       36849    36881      +32     
==========================================
+ Hits        36134    36165      +31     
- Misses        715      716       +1     

Flag	Coverage Δ
integration	98.05% <97.56%> (-0.01%)	⬇️
latest-uploader-overall	98.05% <97.56%> (-0.01%)	⬇️
unit	98.05% <97.56%> (-0.01%)	⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Components	Coverage Δ
NonTestCode	95.91% <95.55%> (+<0.01%)	⬆️
OutsideTasks	98.06% <97.56%> (-0.01%)	⬇️

Files with missing lines	Coverage Δ
...ces/bundle_analysis/new_notify/contexts/comment.py	98.75% <100.00%> (+0.33%)	⬆️
.../new_notify/contexts/tests/test_comment_context.py	100.00% <100.00%> (ø)
...ces/bundle_analysis/new_notify/messages/comment.py	100.00% <100.00%> (ø)
...analysis/new_notify/messages/tests/test_comment.py	100.00% <100.00%> (ø)
services/activation.py	93.10% <60.00%> (-6.90%)	⬇️

... and 6 files with indirect coverage changes

[codecov-public-qa]

Codecov Report

Attention: Patch coverage is 97.56098% with 2 lines in your changes missing coverage. Please review.

Project coverage is 98.05%. Comparing base (aea8367) to head (a607d0a).
Report is 1 commits behind head on main.

✅ All tests successful. No failed tests found.

@@            Coverage Diff             @@
##             main     #654      +/-   ##
==========================================
- Coverage   98.05%   98.05%   -0.01%     
==========================================
  Files         437      437              
  Lines       36849    36881      +32     
==========================================
+ Hits        36134    36165      +31     
- Misses        715      716       +1     

Flag	Coverage Δ
integration	98.05% <97.56%> (-0.01%)	⬇️
latest-uploader-overall	98.05% <97.56%> (-0.01%)	⬇️
unit	98.05% <97.56%> (-0.01%)	⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Components	Coverage Δ
NonTestCode	95.91% <95.55%> (+<0.01%)	⬆️
OutsideTasks	98.06% <97.56%> (-0.01%)	⬇️

Files	Coverage Δ
...ces/bundle_analysis/new_notify/contexts/comment.py	98.75% <100.00%> (+0.33%)	⬆️
.../new_notify/contexts/tests/test_comment_context.py	100.00% <100.00%> (ø)
...ces/bundle_analysis/new_notify/messages/comment.py	100.00% <100.00%> (ø)
...analysis/new_notify/messages/tests/test_comment.py	100.00% <100.00%> (ø)
services/activation.py	93.10% <60.00%> (-6.90%)	⬇️

... and 6 files with indirect coverage changes

[codecov]

Codecov Report

Attention: Patch coverage is 97.56098% with 2 lines in your changes missing coverage. Please review.

Project coverage is 98.10%. Comparing base (aea8367) to head (a607d0a).
Report is 6 commits behind head on main.

Changes have been made to critical files, which contain lines commonly executed in production. Learn more

✅ All tests successful. No failed tests found.

Files with missing lines	Patch %	Lines
services/activation.py	60.00%	2 Missing ⚠️

Additional details and impacted files

@@           Coverage Diff            @@
##             main     #654    +/-   ##
========================================
  Coverage   98.10%   98.10%            
========================================
  Files         476      476            
  Lines       38170    38357   +187     
========================================
+ Hits        37447    37632   +185     
- Misses        723      725     +2     

Flag	Coverage Δ
integration	98.05% <97.56%> (-0.01%)	⬇️
latest-uploader-overall	98.05% <97.56%> (-0.01%)	⬇️
unit	98.05% <97.56%> (-0.01%)	⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Components	Coverage Δ
NonTestCode	96.02% <95.55%> (-0.01%)	⬇️
OutsideTasks	98.06% <97.56%> (-0.01%)	⬇️

Files with missing lines	Coverage Δ
...ces/bundle_analysis/new_notify/contexts/comment.py	98.75% <100.00%> (+0.33%)	⬆️
.../new_notify/contexts/tests/test_comment_context.py	100.00% <100.00%> (ø)
...ces/bundle_analysis/new_notify/messages/comment.py	100.00% <100.00%> (ø)
...analysis/new_notify/messages/tests/test_comment.py	100.00% <100.00%> (ø)
services/activation.py	93.10% <60.00%> (-6.90%)	⬇️

... and 6 files with indirect coverage changes

This change has been scanned for critical changes. Learn more

[Swatinem]

Both of these depend on requires_license(). I would rather define a is_saas: bool field for the template and match on that. In particular, the template already matches on codecov_instance in a bunch of places, but the codecov_name is being resolved in python code instead.