refactor: remove some of the global state, part 1

Mikhail Mazurskiyrequested to merge
ash2k/cleanups into main

Description

The code is really tangled and hard to modify. I've removed some usages of the global client but there are a couple more. I'll get to them a bit later.

Related Issues

Relates to Reduce/remove global mutable state (#7902 - closed).

How has this been tested?

Unit tests pass, but I also want to build it and run a few commands. Haven't done that yet.

Screenshots (if appropriate):

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Documentation
  • Chore (Related to CI or Packaging to platforms)
  • Test gap
Edited by Mikhail Mazurskiy

Merge request reports