Skip to content

feat: switch xmloxide to github backend#229

Merged
zeitlinger merged 2 commits intomainfrom
codex/xmloxide-github-backend
Apr 25, 2026
Merged

feat: switch xmloxide to github backend#229
zeitlinger merged 2 commits intomainfrom
codex/xmloxide-github-backend

Conversation

@zeitlinger
Copy link
Copy Markdown
Member

Summary

Switch xmloxide from the cargo: mise backend to the github: backend now that upstream publishes GitHub releases in v0.4.2.

What Changed

  • changed the canonical xmllint tool key to github:jonwiggins/xmloxide
  • added an obsolete-key migration so flint update rewrites existing cargo:xmloxide entries
  • updated this repo's own mise.toml, Renovate metadata, and xmllint fixtures to the new backend and version

Why

xmloxide release v0.4.2 is now available as a GitHub release artifact, so we can install it through mise's GitHub backend and stop relying on the Cargo backend for this tool.

Impact

Consumers who already use cargo:xmloxide can migrate automatically via flint update.
Newly generated or maintained configs will use the GitHub-backed install path instead.

Validation

  • FLINT_CASES=xmllint cargo test -q cases
  • cargo test -q
  • mise run lint:fix
  • mise run lint

@zeitlinger zeitlinger changed the title [codex] fix: switch xmloxide to github backend feat: switch xmloxide to github backend Apr 24, 2026
…-backend

# Conflicts:
#	.github/renovate-tracked-deps.json
#	src/registry/tests.rs
@zeitlinger zeitlinger marked this pull request as ready for review April 24, 2026 17:14
@zeitlinger zeitlinger requested a review from a team as a code owner April 24, 2026 17:14
Copilot AI review requested due to automatic review settings April 24, 2026 17:14
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Switches the xmloxide install source used for the xmllint check from mise’s cargo: backend to the github: backend now that upstream publishes GitHub releases, and wires in an automatic migration for existing configs.

Changes:

  • Updated the xmllint registry check to use github:jonwiggins/xmloxide.
  • Added an obsolete-key rewrite mapping so flint update migrates cargo:xmloxidegithub:jonwiggins/xmloxide.
  • Updated repo fixtures/config and Renovate tracking to the new tool key + version (v0.4.2).

Reviewed changes

Copilot reviewed 8 out of 8 changed files in this pull request and generated no comments.

Show a summary per file
File Description
src/registry/checks.rs Switches xmllint’s mise tool key to the GitHub backend.
src/registry/obsolete.rs Adds the cargo:xmloxidegithub:jonwiggins/xmloxide migration mapping.
src/registry/tests.rs Updates registry-key assertions to reflect the new canonical key.
mise.toml Updates this repo’s pinned tool entry to the new GitHub-backed xmloxide version.
default.json Updates Renovate’s weekly “linters” package list to match the registry key change.
.github/renovate-tracked-deps.json Updates the tracked mise tool key for Renovate metadata.
tests/cases/xmllint/clean/files/mise.toml Updates xmllint fixture to use the new tool key/version.
tests/cases/xmllint/failure/files/mise.toml Updates xmllint fixture to use the new tool key/version.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@zeitlinger zeitlinger merged commit aaf59e7 into main Apr 25, 2026
17 checks passed
@zeitlinger zeitlinger deleted the codex/xmloxide-github-backend branch April 25, 2026 09:15
This was referenced Apr 25, 2026
zeitlinger added a commit that referenced this pull request Apr 27, 2026
### Added

- add flint setup preflight migrations
([#233](#233))
- support managed linter configs and aqua-backed tool keys
([#228](#228))
- switch xmloxide to github backend
([#229](#229))
- add taplo formatter check
([#224](#224))
- *(init)* configure line length during init
([#218](#218))

### Fixed

- support renovate v43.136.3
([#223](#223))
- switch yaml-lint to ryl
([#226](#226))
- *(ruff)* install via github releases
([#227](#227))
- resolve init rollout regressions
([#220](#220))

### Other

- reduce fake-bin fixture coverage
([#240](#240))
- *(deps)* update dependency npm:renovate to v43.141.1
([#221](#221))
- guard fixture case groups
([#238](#238))
- *(deps)* update dependency grafana/flint to v0.20.4
([#235](#235))
- *(deps)* lock file maintenance
([#237](#237))
- *(deps)* update dependency mise to v2026.4.19
([#236](#236))
- *(deps)* update rust crate json5 to v1
([#232](#232))
- *(deps)* update taiki-e/install-action digest to 787505c
([#219](#219))
- *(deps)* update dependency grafana/flint to v0.20.3
([#225](#225))

> [!IMPORTANT]
> Close and reopen this PR to trigger CI checks.

---------

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: Gregor Zeitlinger <[email protected]>
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.

3 participants