Prepare platform-anvil-prod (#5015, #4031, #4032, #4177, #5077) #5184
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
![github-advanced-security[bot]](https://avatars.githubusercontent.com/in/57789?s=60&v=4){kind=small}
|
|
||
| def bucket_regional_domain_name(bucket): | ||
| if False: | ||
| return '${aws_s3_bucket.%s.bucket_regional_domain_name}' % bucket # noqa |
Check warning
Code scanning / CodeQL
Unreachable code
Codecov Report
@@ Coverage Diff @@
## develop #5184 +/- ##
===========================================
- Coverage 84.41% 83.93% -0.49%
===========================================
Files 149 150 +1
Lines 18305 18427 +122
===========================================
+ Hits 15452 15466 +14
- Misses 2853 2961 +108
|
hannes-ucsc
force-pushed
the
issues/hannes-ucsc/5015-anvilprod
branch
8 times, most recently
from
04c1d3b to
5bd3332
Compare
hannes-ucsc
force-pushed
the
issues/hannes-ucsc/5015-anvilprod
branch
5 times, most recently
from
7dc7efd to
4fe493e
Compare
14 tasks
hannes-ucsc
force-pushed
the
issues/hannes-ucsc/5015-anvilprod
branch
3 times, most recently
from
f5f2173 to
249074f
Compare
14 tasks
hannes-ucsc
force-pushed
the
issues/hannes-ucsc/5015-anvilprod
branch
2 times, most recently
from
0abed26 to
6dba204
Compare
88 tasks
hannes-ucsc
force-pushed
the
issues/hannes-ucsc/5015-anvilprod
branch
from
6dba204 to
e2e3e59
Compare
hannes-ucsc
changed the title
… as that magically fixes some type warnings in PyCharm
No semantic change
… that now causes a type warning
hannes-ucsc
force-pushed
the
issues/hannes-ucsc/5015-anvilprod
branch
from
b04702d to
238b543
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Connected issues: #5015, #4031, #4032, #4177, #5077
Checklist
Author
developissues/<GitHub handle of author>/<issue#>-<slug>partiallabel to PR or this PR completely resolves all connected issues1 when the issue title describes a problem, the corresponding PR
title is
Fix:followed by the issue titleAuthor (reindex, API changes)
rtag to commit title or this PR does not require reindexingreindexlabel to PR or this PR does not require reindexinga(compatible changes) orA(incompatible ones) tag to commit title or this PR does not modify the Azul service APIAPIlabel to connected issues or this PR does not modify the Azul service APIAuthor (chains)
baselabel to the blocking PR or this PR is not chained to another PRchainedlabel to this PR or this PR is not chained to another PRAuthor (upgrading)
utag to commit title or this PR does not require upgradingupgradelabel to PR or this PR does not require upgradingAuthor (operator tasks)
Author (hotfixes)
Ftag to main commit title or this PR does not include permanent fix for a temporary hotfixprodbranch has no temporary hotfixes for any connected issuesAuthor (before every review)
develop, squashed old fixupsmake requirements_updateor this PR does not touch requirements*.txt, common.mk, Makefile and DockerfileRtag to commit title or this PR does not touch requirements*.txtreqslabel to PR or this PR does not touch requirements*.txtmake integration_testpasses in personal deployment or this PR does not touch functionality that could break the ITPeer reviewer (after requesting changes)
Uncheck the Author (before every review) checklists.
Peer reviewer (after approval)
Primary reviewer (after requesting changes)
Uncheck the before every review checklists. Update the
N reviewslabel.Primary reviewer (after approval)
demoorno demono demono sandboxN reviewslabel is accurateOperator (before pushing merge the commit)
shared.devgitlab.devshared.anvildevgitlab.anvildevshared.anvilprodcomponentgitlab.anvilprodcomponentreindexlabel andrcommit title tagno demoupgradedevelopdevand addedsandboxlabel or PR is labeledno sandboxanvildevor PR is labeledno sandboxsandboxdeployment or PR is labeledno sandboxanvilboxdeployment or PR is labeledno sandboxsandboxdeployment or PR is labeledno sandboxanvilboxdeployment or PR is labeledno sandboxsandboxor this PR does not remove catalogs or otherwise causes unreferenced indicesanvilboxor this PR does not remove catalogs or otherwise causes unreferenced indicessandboxor this PR does not require reindexingsandboxanvilboxor this PR does not require reindexingsandboxsandboxor this PR does not require reindexingsandboxanvilboxor this PR does not require reindexingsandboxOperator (chain shortening)
developor this PR is not labeledbasechainedlabel from the blocked PR or this PR is not labeledbasebasebaselabel from this PR or this PR is not labeledbaseOperator (after pushing the merge commit)
devor PR is labeledno sandboxanvildevor PR is labeledno sandboxanvilproddev1dev1anvildev1anvildev1anvilprod1anvilprod1devanvildevanvilprod1 When pushing the merge commit is skipped due to the PR being
labelled
no sandbox, the next build triggered by a PR whose merge commit ispushed determines this checklist item.
Operator (reindex)
devor this PR does not remove catalogs or otherwise causes unreferenced indicesanvildevor this PR does not remove catalogs or otherwise causes unreferenced indicesanvilprodor this PR does not remove catalogs or otherwise causes unreferenced indicesdevor this PR does not require reindexinganvildevor this PR does not require reindexinganvilprodor this PR does not require reindexingdevor this PR does not require reindexinganvildevor this PR does not require reindexinganvilprodor this PR does not require reindexingdevdeployment or this PR does not require reindexinganvildevdeployment or this PR does not require reindexinganvilproddeployment or this PR does not require reindexingOperator
Shorthand for review comments
Lline is too longWline wrapping is wrongQbad quotesFother formatting problem