fix: update get_ta_url to handle null branch#1181
Conversation
there's a deadlock error that happens where the timescale background worker is running background jobs and we're altering the schema of the tables on which it's running the background job on, so we stop the background worker momentarily to allow the migrations to run.
✅ Sentry found no issues in your recent changes ✅ |
Codecov ReportAttention: Patch coverage is ✅ All tests successful. No failed tests found.
📢 Thoughts on this report? Let us know! |
Codecov ReportAttention: Patch coverage is
✅ All tests successful. No failed tests found.
Additional details and impacted files@@ Coverage Diff @@
## main #1181 +/- ##
==========================================
- Coverage 97.72% 97.72% -0.01%
==========================================
Files 453 455 +2
Lines 37117 37157 +40
==========================================
+ Hits 36273 36311 +38
- Misses 844 846 +2
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
| if commit.branch is not None: | ||
| branch_name = quote_plus(commit.branch) | ||
| else: | ||
| branch_name = "main" |
There was a problem hiding this comment.
as you have access to Repository, please pick the main branch as it is stored in repo. (and also push that through quote_plus)
7d960fd to
2765ece
Compare
No description provided.