Skip to content

Comments

test: Move ruff to tool stub and fix up tests#487

Merged
jdx merged 9 commits intojdx:mainfrom
thejcannon:jcannon/tool-stub-ruff
Dec 5, 2025
Merged

test: Move ruff to tool stub and fix up tests#487
jdx merged 9 commits intojdx:mainfrom
thejcannon:jcannon/tool-stub-ruff

Conversation

@thejcannon
Copy link
Contributor

@thejcannon thejcannon commented Dec 4, 2025

  • Moves ruff from root mise.toml into tool-stub
  • slight refactor to the tests

Note

Moves ruff from mise config to a tool stub, updates ruff steps to force-exclude, and refactors stdin-related tests away from ruff format.

  • Tooling:
    • Move ruff from mise.toml/mise.lock into test/builtin_tool_stubs/ruff (v0.13.3).
  • PKL builtins:
    • pkl/builtins/ruff.pkl: use ruff check --force-exclude.
    • pkl/builtins/ruff.pkl and pkl/builtins/ruff_format.pkl: simplify tests with bad/good fixtures and split check/fix cases.
  • Tests:
    • test/stdin.bats: switch from ruff format to hk util trailing-whitespace with xargs/prefix flows; remove atfiles /dev/stdin case; update expectations (use cat -e, trailing $).

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

@jdx
Copy link
Owner

jdx commented Dec 4, 2025

bugbot run

Copy link

@cursor cursor bot left a comment

Choose a reason for hiding this comment

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

✅ Bugbot reviewed your changes and found no bugs!


@jdx
Copy link
Owner

jdx commented Dec 4, 2025

this looks like a mise bug, I'm investigating

@thejcannon
Copy link
Contributor Author

Hmm "works on my machine". I wonder if the version of mise in CI needs bumping?

@thejcannon
Copy link
Contributor Author

BTW @jdx I think we're seeing the mise concurrency bug in CI:

@thejcannon
Copy link
Contributor Author

But the latest failure seems legit (I didn't look too hard, but I'm assuming there's some other .bats test that's using ruff)

@thejcannon thejcannon force-pushed the jcannon/tool-stub-ruff branch from 362fa74 to e969841 Compare December 5, 2025 03:03
@thejcannon
Copy link
Contributor Author

@jdx alright, should be good now

@jdx
Copy link
Owner

jdx commented Dec 5, 2025

bugbot run

Copy link

@cursor cursor bot left a comment

Choose a reason for hiding this comment

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

✅ Bugbot reviewed your changes and found no bugs!


@jdx jdx merged commit cd473df into jdx:main Dec 5, 2025
15 checks passed
@jdx jdx mentioned this pull request Dec 5, 2025
@thejcannon thejcannon deleted the jcannon/tool-stub-ruff branch December 5, 2025 11:45
jdx added a commit that referenced this pull request Dec 12, 2025
## [1.27.0](https://github.com/jdx/hk/compare/v1.26.0..v1.27.0) -
2025-12-12

### 🚀 Features

- **(lychee)** new builtin by [@scop](https://github.com/scop) in
[#510](#510)
- **(tombi)** add tombi config to hk builtin config by
[@hituzi-no-sippo](https://github.com/hituzi-no-sippo) in
[#511](#511)
- Apply the step glob to the test files fallback by
[@thejcannon](https://github.com/thejcannon) in
[#498](#498)
- Support `hk.local.pkl` by [@thejcannon](https://github.com/thejcannon)
in [#504](#504)
- add yamlfmt config to hk builtin config by
[@hituzi-no-sippo](https://github.com/hituzi-no-sippo) in
[#505](#505)
- add `HK_PKL_HTTP_REWRITE` envvar by
[@thejcannon](https://github.com/thejcannon) in
[#512](#512)

### 🐛 Bug Fixes

- **(builtins)** use `stage = "<JOB_FILES>"` for fix_smart_quotes
builtin by [@joonas](https://github.com/joonas) in
[#503](#503)
- add new mise builtin to Builtins.pkl by
[@hisaac](https://github.com/hisaac) in
[#492](#492)
- preserve file permissions in fix-smart-quotes util by
[@joonas](https://github.com/joonas) in
[#506](#506)

### 🚜 Refactor

- Turn on `taplo-format` by [@thejcannon](https://github.com/thejcannon)
in [#501](#501)

### 📚 Documentation

- Generate config docs from the pkl by
[@thejcannon](https://github.com/thejcannon) in
[#499](#499)

### 🧪 Testing

- Move `ruff` to tool stub and fix up tests by
[@thejcannon](https://github.com/thejcannon) in
[#487](#487)
- Do some test gardening by [@thejcannon](https://github.com/thejcannon)
in [#497](#497)
- Clean up `test/pkl_config_errors.bats` by
[@thejcannon](https://github.com/thejcannon) in
[#500](#500)
- add yamllint tests by [@thejcannon](https://github.com/thejcannon) in
[#509](#509)

### 📦️ Dependency Updates

- update swatinem/rust-cache digest to 779680d by
[@renovate[bot]](https://github.com/renovate[bot]) in
[#495](#495)
- update anthropics/claude-code-action digest to 6337623 by
[@renovate[bot]](https://github.com/renovate[bot]) in
[#494](#494)
- lock file maintenance by
[@renovate[bot]](https://github.com/renovate[bot]) in
[#502](#502)

### New Contributors

- @hituzi-no-sippo made their first contribution in
[#511](#511)

<!-- CURSOR_SUMMARY -->
---

> [!NOTE]
> Bumps project to v1.27.0, updates docs/examples and CLI metadata,
refreshes init template and changelog, and updates dependency lockfile.
> 
> - **Release/versioning**
> - Bump version to `1.27.0` in `Cargo.toml`, `hk.usage.kdl`,
`docs/cli/commands.json`, and across docs/examples (`amends`/`import`
URLs).
>   - Add `1.27.0` section to `CHANGELOG.md`.
> - **Docs**
> - Update pkl import references to `v1.27.0` throughout `docs/*` and
`pkl/Config.pkl` docstrings.
> - **CLI/init**
>   - Update `src/cli/init.rs` template to reference `v1.27.0`.
>   - Adjust error hint in `src/config.rs` to new versioned URL.
> - **Dependencies**
> - Refresh `Cargo.lock` with crate updates (e.g., `reqwest`, `git2`,
`libgit2-sys`, `tower-http`, compression libs).
> 
> <sup>Written by [Cursor
Bugbot](https://cursor.com/dashboard?tab=bugbot) for commit
342dc3f. This will update automatically
on new commits. Configure
[here](https://cursor.com/dashboard?tab=bugbot).</sup>
<!-- /CURSOR_SUMMARY -->

Co-authored-by: mise-en-dev <[email protected]>
tmeijn pushed a commit to tmeijn/dotfiles that referenced this pull request Dec 16, 2025
This MR contains the following updates:

| Package | Update | Change |
|---|---|---|
| [hk](https://github.com/jdx/hk) | minor | `1.26.0` -> `1.27.0` |

MR created with the help of [el-capitano/tools/renovate-bot](https://gitlab.com/el-capitano/tools/renovate-bot).

**Proposed changes to behavior should be submitted there as MRs.**

---

### Release Notes

<details>
<summary>jdx/hk (hk)</summary>

### [`v1.27.0`](https://github.com/jdx/hk/blob/HEAD/CHANGELOG.md#1270---2025-12-12)

[Compare Source](jdx/hk@v1.26.0...v1.27.0)

##### 🚀 Features

- **(lychee)** new builtin by [@&#8203;scop](https://github.com/scop) in [#&#8203;510](jdx/hk#510)
- **(tombi)** add tombi config to hk builtin config by [@&#8203;hituzi-no-sippo](https://github.com/hituzi-no-sippo) in [#&#8203;511](jdx/hk#511)
- Apply the step glob to the test files fallback by [@&#8203;thejcannon](https://github.com/thejcannon) in [#&#8203;498](jdx/hk#498)
- Support `hk.local.pkl` by [@&#8203;thejcannon](https://github.com/thejcannon) in [#&#8203;504](jdx/hk#504)
- add yamlfmt config to hk builtin config by [@&#8203;hituzi-no-sippo](https://github.com/hituzi-no-sippo) in [#&#8203;505](jdx/hk#505)
- add `HK_PKL_HTTP_REWRITE` envvar by [@&#8203;thejcannon](https://github.com/thejcannon) in [#&#8203;512](jdx/hk#512)

##### 🐛 Bug Fixes

- **(builtins)** use `stage = "<JOB_FILES>"` for fix\_smart\_quotes builtin by [@&#8203;joonas](https://github.com/joonas) in [#&#8203;503](jdx/hk#503)
- add new mise builtin to Builtins.pkl by [@&#8203;hisaac](https://github.com/hisaac) in [#&#8203;492](jdx/hk#492)
- preserve file permissions in fix-smart-quotes util by [@&#8203;joonas](https://github.com/joonas) in [#&#8203;506](jdx/hk#506)

##### 🚜 Refactor

- Turn on `taplo-format` by [@&#8203;thejcannon](https://github.com/thejcannon) in [#&#8203;501](jdx/hk#501)

##### 📚 Documentation

- Generate config docs from the pkl by [@&#8203;thejcannon](https://github.com/thejcannon) in [#&#8203;499](jdx/hk#499)

##### 🧪 Testing

- Move `ruff` to tool stub and fix up tests by [@&#8203;thejcannon](https://github.com/thejcannon) in [#&#8203;487](jdx/hk#487)
- Do some test gardening by [@&#8203;thejcannon](https://github.com/thejcannon) in [#&#8203;497](jdx/hk#497)
- Clean up `test/pkl_config_errors.bats` by [@&#8203;thejcannon](https://github.com/thejcannon) in [#&#8203;500](jdx/hk#500)
- add yamllint tests by [@&#8203;thejcannon](https://github.com/thejcannon) in [#&#8203;509](jdx/hk#509)

##### 📦️ Dependency Updates

- update swatinem/rust-cache digest to [`779680d`](jdx/hk@779680d) by [@&#8203;renovate\[bot\]](https://github.com/renovate\[bot]) in [#&#8203;495](jdx/hk#495)
- update anthropics/claude-code-action digest to [`6337623`](jdx/hk@6337623) by [@&#8203;renovate\[bot\]](https://github.com/renovate\[bot]) in [#&#8203;494](jdx/hk#494)
- lock file maintenance by [@&#8203;renovate\[bot\]](https://github.com/renovate\[bot]) in [#&#8203;502](jdx/hk#502)

##### New Contributors

- [@&#8203;hituzi-no-sippo](https://github.com/hituzi-no-sippo) made their first contribution in [#&#8203;511](jdx/hk#511)

</details>

---

### Configuration

📅 **Schedule**: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).

🚦 **Automerge**: Disabled by config. Please merge this manually once you are satisfied.

♻ **Rebasing**: Whenever MR becomes conflicted, or you tick the rebase/retry checkbox.

🔕 **Ignore**: Close this MR and you won't be reminded about this update again.

---

 - [ ] <!-- rebase-check -->If you want to rebase/retry this MR, check this box

---

This MR has been generated by [Renovate Bot](https://github.com/renovatebot/renovate).
<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiI0Mi40OC4wIiwidXBkYXRlZEluVmVyIjoiNDIuNDguMCIsInRhcmdldEJyYW5jaCI6Im1haW4iLCJsYWJlbHMiOlsiUmVub3ZhdGUgQm90Il19-->
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