Skip to content

clean up some imports - #1472

Merged
prandla merged 2 commits into
cms-dev:mainfrom
e-i-o:import-cleanup
Jul 23, 2025
Merged

clean up some imports#1472
prandla merged 2 commits into
cms-dev:mainfrom
e-i-o:import-cleanup

Conversation

@prandla

@prandla prandla commented Jul 22, 2025

Copy link
Copy Markdown
Member
  • Remove unneeded import attempts from "tornado4". From what I can tell, this was only ever useful when using system packages on ubuntu 20.04. We don't support installation using system packages anymore (and ubuntu 20.04 is EOL).
  • Remove misleading comment about monkey-patching the DB connection string in the test suite. We don't monkey-patch it anymore (instead we assume someone else fixed it and assert that it's correct).

* Remove unneeded import attempts from "tornado4". From what I can tell,
  this was only ever useful when using system packages on ubuntu 20.04.
  We don't support installation using system packages anymore (and
  ubuntu 20.04 is EOL).
* Remove misleading comment about monkey-patching the DB connection
  string in the test suite. We don't monkey-patch it anymore (instead we
  assume someone else fixed it and assert that it's correct).
@prandla
prandla merged commit b1b2e95 into cms-dev:main Jul 23, 2025
3 checks passed
@prandla
prandla deleted the import-cleanup branch July 23, 2025 14:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants