eth3lbert

Results 51 comments of eth3lbert

Based on the [CI failure](https://github.com/astral-sh/uv/actions/runs/10442002933/job/28913829027#step:8:14394) ``` running 1 test ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ Snapshot Summary ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ Snapshot: tool_install_version-3 Source: crates/uv/tests/tool_install.rs:306 ──────────────────────────────────────────────────────────────────────────────── Expression: fs_err::read_to_string(tool_dir.join("black").join("uv-receipt.toml")).unwrap() ──────────────────────────────────────────────────────────────────────────────── -old snapshot +new results ────────────┬─────────────────────────────────────────────────────────────────── 4 4 │ {...

Also note that the default value varies by platform. We might need some help text about this to prevent confusion when reading the cli reference on the site.

I'm not sure how we should handle the snapshot of cli references, as the output depends on the platform it's generated on. :sweat_smile:

Or perhaps for this kind of platform-dependent default value, it would be better to display something similar to the CLI reference? e.g. ``` shell-session --link-mode The method to use when...

Based on the CI feedback, resetting only for `ToolOptions` seems more correct. Now, it only fails on cli references :)

I also noticed that we could implement the `Display` for `LinkMode` to allow setting the default value via `default_value = LinkMode::default().to_string()`. This way, we wouldn't need to change the type....

> As more reference, I had to undo the default value in #6835 > > We don't have access to `ArgMatches`, though maybe we could refactor things so we do....

You should be able to make it work by changing `Option` to `LinkMode` and setting `default_value_t` to `LinkMode::default()`. ``` --link-mode The method to use when installing packages from the global...

Yes, I've heard a lot about complaints regarding bazel's complexity.

Is disabling ssl verification an option?