Skip to content

Limit mutating global state#233

Merged
jaraco merged 9 commits intomainfrom
quality/231-global-state
Mar 2, 2024
Merged

Limit mutating global state#233
jaraco merged 9 commits intomainfrom
quality/231-global-state

Conversation

@jaraco
Copy link
Copy Markdown
Member

@jaraco jaraco commented Mar 2, 2024

  • Extract a method for customizing the compiler for macOS.
  • Convert comment to docstring; update wording.
  • Create a fixture to patch-out compiler customization on macOS.
  • Utilize the fixture for disabling compiler customization on macOS for cxx test. Closes Test suite is brittle around global sysconfig._config_vars state. #231.
  • Limit mutating global state and simply rely on functools.lru_cache to limit the behavior to a single invocation.

@jaraco jaraco force-pushed the quality/231-global-state branch from b853cc3 to 9e83319 Compare March 2, 2024 15:34
@jaraco jaraco merged commit 5b74c86 into main Mar 2, 2024
@jaraco jaraco deleted the quality/231-global-state branch March 2, 2024 15:36
jaraco added a commit that referenced this pull request Mar 2, 2024
@jaraco jaraco restored the quality/231-global-state branch March 2, 2024 15:38
@jaraco jaraco deleted the quality/231-global-state branch March 2, 2024 15:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Test suite is brittle around global sysconfig._config_vars state.

1 participant