Skip to content

fix(config): top-level 'concurrency' key incorrectly overwrote pypi-c… - #6665

Draft
Hillindigo wants to merge 2 commits into
prefix-dev:mainfrom
Hillindigo:fix/config-concurrency-overwrites-pypi
Draft

fix(config): top-level 'concurrency' key incorrectly overwrote pypi-c…#6665
Hillindigo wants to merge 2 commits into
prefix-dev:mainfrom
Hillindigo:fix/config-concurrency-overwrites-pypi

Conversation

@Hillindigo

@Hillindigo Hillindigo commented Jul 24, 2026

Copy link
Copy Markdown

…onfig

The 'concurrency' branch in Config::set was copy-pasted from the 'pypi-config' branch without updating the target field. Setting pixi config set concurrency ... silently overwrote pypi_config instead of concurrency settings, causing data loss of PyPI index URL, keyring provider, and other PyPI configuration.

Added regression test to verify that setting/unsetting the top-level 'concurrency' key correctly updates concurrency fields and leaves pypi_config untouched.

Description

Fixes #{issue}

How Has This Been Tested?

AI Disclosure

  • This PR contains AI-generated content.
    • I have tested any AI-generated content in my PR.
    • I take responsibility for any AI-generated content in my PR.

Tools: {e.g., Claude, Codex, GitHub Copilot, ChatGPT, etc.}

Checklist:

  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • I have added sufficient tests to cover my changes.
  • I have verified that changes that would impact the JSON schema have been made in schema/model.py.

hp added 2 commits July 24, 2026 17:53
…onfig

The 'concurrency' branch in Config::set was copy-pasted from the
'pypi-config' branch without updating the target field. Setting
pixi config set concurrency ... silently overwrote pypi_config
instead of concurrency settings, causing data loss of PyPI index
URL, keyring provider, and other PyPI configuration.

Added regression test to verify that setting/unsetting the
top-level 'concurrency' key correctly updates concurrency fields
and leaves pypi_config untouched.
@ruben-arts

Copy link
Copy Markdown
Contributor

Please cleanup the PR description before we can review it.

@ruben-arts
ruben-arts marked this pull request as draft July 27, 2026 07:54
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.

2 participants