Skip to content

Conversation

@jylenhof
Copy link
Contributor

@jylenhof jylenhof commented Nov 16, 2025

Note

Adds kubeswitch to the tools registry and introduces a version test for ruff.

  • Tools Registry:
    • New tool: kubeswitch
      • Backends: aqua:danielfoehrKn/kubeswitch, github:danielfoehrKn/kubeswitch
      • Description and test added (switcher --version expecting switcher version v{{version}}).
    • Update: ruff
      • Added test to validate version output (ruff --version expecting ruff {{version}}).

Written by Cursor Bugbot for commit 61181ff. This will update automatically on new commits. Configure here.

@jylenhof jylenhof changed the title feat(registry): add rumdl and kubeswitch tools registry: add rumdl and kubeswitch tools Nov 16, 2025
@jdx
Copy link
Owner

jdx commented Nov 17, 2025

kubeswitch yes, rumdl no

@jylenhof
Copy link
Contributor Author

Can you explain your rational about rumdl ?

Maybe a little younger ? I think it could replace soon markdownlint, and it propose 'format' in addition to 'lint' not offered by markdownlint if I'm not wrong

For now I updated the PR without it

@jylenhof jylenhof changed the title registry: add rumdl and kubeswitch tools registry: add kubeswitch tool Nov 17, 2025
@jylenhof jylenhof changed the title registry: add kubeswitch tool registry: add kubeswitch tool and add test for ruff Nov 17, 2025
@jdx jdx enabled auto-merge (squash) November 18, 2025 07:05
description = "Tools for observing Kubernetes resources in real time, powered by Pulumi"

[tools.kubeswitch]
backends = ["aqua:danielfoehrKn/kubeswitch", "github:danielfoehrKn/kubeswitch"]
Copy link

Choose a reason for hiding this comment

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

Bug: Binary Name Mismatch Breaks Backend Tests

The github:danielfoehrKn/kubeswitch backend is missing the exe=switcher parameter to specify the correct binary name. The test expects switcher command, but github backend will produce kubeswitch binary by default, causing the test to fail.

Fix in Cursor Fix in Web

@jdx jdx merged commit 5484fdb into jdx:main Nov 18, 2025
34 of 38 checks passed
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