Adrian Edwards
Adrian Edwards
Closing. I dont see a path to these specific changes getting merged in this specific PR. Discussion should happen in other issues/PRs addressing the repo move detection, such as #3391
> @MoralCode Sorry for this question, but if possible can you make a small explanation what it means by tech debt? I can't understand what does this means. I'll probably...
> @MoralCode the testcase which is failing is showing that in some files `else` is used after `return` and that's where around 9-10 warnings are there which is failing `run-linting-checks/runner/pylint...
I think this is ready but im holding off on marking it as such until we get the next release cut
oops, i should probably test this soon
I really dont think thats the answer
My plan: 1. Wait for database stuff to be fixed/merged 2. Fix database test fixtures in unit tests 3. Create unit test confirming one connection is leaked every time the...
OK so it seems like the smoke tests revealed an interesting bug Stack Trace ``` augur-1 | Traceback (most recent call last): augur-1 | File "/augur/.venv/bin/augur", line 10, in augur-1...
> 2\. Also, might be worth validating no duplicates exist for `gh_login` and `cntrb_login` before the unique constraints get added - especially on systems with manually altered DBs. Does alembic...
Added schemas to sequences. Also migrated the operations and spdx schemas to use these newer sequence calls too. gave the duplication check queries a run on our instance and.... we...