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

feat: Emit sync repo message to Shelter on repo update #215

Merged
merged 2 commits into from
Nov 6, 2023

Conversation

scott-codecov
Copy link
Contributor

@codecov-staging
Copy link

codecov-staging bot commented Oct 27, 2023

Codecov Report

All modified and coverable lines are covered by tests ✅

📢 Thoughts on this report? Let us know!

@codecov-qa
Copy link

codecov-qa bot commented Oct 27, 2023

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (93891cd) 95.66% compared to head (d540684) 95.67%.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #215   +/-   ##
=======================================
  Coverage   95.66%   95.67%           
=======================================
  Files         603      604    +1     
  Lines       15435    15458   +23     
=======================================
+ Hits        14766    14789   +23     
  Misses        669      669           
Flag Coverage Δ
unit 95.67% <100.00%> (+<0.01%) ⬆️
unit-latest-uploader 95.67% <100.00%> (+<0.01%) ⬆️

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

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@codecov-public-qa
Copy link

codecov-public-qa bot commented Oct 27, 2023

Codecov Report

❗ No coverage uploaded for pull request base (main@93891cd). Click here to learn what that means.
The diff coverage is 100.00%.

❗ Current head f544074 differs from pull request most recent head d540684. Consider uploading reports for the commit d540684 to get more accurate results

Impacted file tree graph

@@           Coverage Diff           @@
##             main     #215   +/-   ##
=======================================
  Coverage        ?   95.62%           
=======================================
  Files           ?      603           
  Lines           ?    15302           
  Branches        ?        0           
=======================================
  Hits            ?    14632           
  Misses          ?      670           
  Partials        ?        0           
Flag Coverage Δ
unit 95.62% <100.00%> (?)
unit-latest-uploader 95.62% <100.00%> (?)

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

Files Coverage Δ
codecov/settings_base.py 85.47% <ø> (ø)
core/apps.py 100.00% <100.00%> (ø)
core/signals.py 100.00% <100.00%> (ø)

Impacted file tree graph

@codecov
Copy link

codecov bot commented Oct 27, 2023

Codecov Report

Merging #215 (f544074) into main (5ddb063) will decrease coverage by 0.01%.
Report is 1 commits behind head on main.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##            main    #215     +/-   ##
=======================================
- Coverage   95.51   95.50   -0.01     
=======================================
  Files        716     717      +1     
  Lines      15673   15952    +279     
=======================================
+ Hits       14970   15234    +264     
- Misses       703     718     +15     
Flag Coverage Δ
unit 95.62% <100.00%> (+0.01%) ⬆️
unit-latest-uploader 95.62% <100.00%> (+0.01%) ⬆️

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

Files Coverage Δ
codecov/settings_base.py 88.39% <ø> (+0.32%) ⬆️
core/apps.py 100.00% <100.00%> (ø)
core/signals.py 100.00% <100.00%> (ø)

... and 5 files with indirect coverage changes

* main:
  Test ats run (#229)
  Release 23.11.2 (#228)
  feat: Add comparison endpoint for impacted files (#212)
  Update shared version (#226)
  feat: add a filter for public/private repo filter for viewableRepositories (#224)
  fix: speedup flag filtering in GraphQL API (#223)
  fix: Disable early notify comment (#219)
  Add 'value' key to the plan representation resolver  (#216)
  713 adjust available plans in plan service (#214)
  feat: Allow upload requests originating from Shelter to set storage path (#172)
@scott-codecov scott-codecov merged commit 6434b23 into main Nov 6, 2023
14 of 15 checks passed
@scott-codecov scott-codecov deleted the scott/cloud-pubsub branch November 6, 2023 16:02
trent-codecov added a commit that referenced this pull request Nov 14, 2023
* Test ats run (#229)

* Test ats run

* Test ats run

* Test ats run

* Test ats run

* Test ats run

* Test ats run

* Test ats run

* Test ats run

* feat: Emit sync repo message to Shelter on repo update (#215)

* feat: add prometheus instrumentation to requests, db accesses, and models (#202)

* feat: add prometheus instrumentation to requests, db accesses, and models

* switch to multiprocessing mode

* add nullchecking for gunicorn worker/pid

* only call multiprocess.mark_process_dead if prometheus multiprocess is enabled

* add comment about public prometheus route

---------

Co-authored-by: Trent Schmidt <trent@codecov.io>

* fix: make GraphQL types matches nullability with django models (#220)


Audit all the GraphQL API types to make sure that the nullabilitity (nullable or not nullable) matches that of the database columns (via Django ORM).

* feat: Add index on pulls (repoid, id) (#217)

* feat: Add index on pulls (repoid, id)

* Fix migration conflict

* Fix XSS vulnerability by removing ref_type from Response (#211)

* fix: available plans for users while trialing (#731) (#230)


Fix a bug where a user while trialing does not have Team plans available to upgrade to.
This happens because while trialing we set plan_user_count to 1000 while trialing, so we will check plan_activated_users while user is trialing.

* fix: Skip login if GitHub OAuth does not return access_token (#236)

* Send github oauth access token expiry with cookie (#237)

* initial commit

* remove method

* lint

* sort imports

* reintroduce variable

* Update small badge (#238)

* chore: Moves the small badge mask over to leave space for 100% while still looking good with 2 digits

* fix small badge expected svg util

* fix small badge tests

* fix: Upgrade Django version to fix security vulnerabilities (#239)


Fixes Django security vulnerabilities by updating to the fix version:

Django Denial-of-service in django.utils.text.Truncator
Django potential denial of service vulnerability in UsernameField on Windows
Django Denial of service vulnerability in django.utils.encoding.uri_to_iri

* feat: Modify billing for Team plan (#637) (#225)


- When signing up to a new paid plan, it will have Team plan choice, and checks for plan user limit.
- Also update limit from 1000 to 2500 and refactor a variable name
- Add isPublic option to measurements filter
- Change marketing name of Pro Team to Pro

* fix: Handle Shelter storage paths in upload download handler (#234)

* fix: Handle shetler storage paths in upload download handler

* fix: Handle shetler storage paths in upload download handler

* Add test for shelter storage path

* Updating workflows (#242)

* Fix urllib and opentelemetry-instrumentation dependencies (#213)

* Update urllib3 dependency
* Fix opentelementry-instrumentation dependency

Fixes: codecov/internal-issues#103
Fixes: codecov/internal-issues#104

Signed-off-by: joseph-sentry <joseph.sawaya@sentry.io>

* Make migration info log less confusing (#221)

Clearly state whether migrations failed or succeded.

Co-authored-by: Trent Schmidt <trent@codecov.io>

* Use alpine 3.18

---------

Signed-off-by: joseph-sentry <joseph.sawaya@sentry.io>
Co-authored-by: scott-codecov <scott@codecov.io>
Co-authored-by: matt-codecov <137832199+matt-codecov@users.noreply.github.com>
Co-authored-by: JerrySentry <142266253+JerrySentry@users.noreply.github.com>
Co-authored-by: joseph-sentry <136376984+joseph-sentry@users.noreply.github.com>
Co-authored-by: Rohit Vinnakota <148245014+rohitvinnakota-codecov@users.noreply.github.com>
Co-authored-by: Terry <87824812+terry-codecov@users.noreply.github.com>
Co-authored-by: Andreas Bergmeier <51448674+AndreasBergmeier6176@users.noreply.github.com>
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.

None yet

2 participants