Skip to content

Commit

Permalink
Remove duplicate warning for settings discovery errors (#7384)
Browse files Browse the repository at this point in the history
## Summary

These are often (but not always) duplicated, and I find it really janky.
This PR makes them tracing warnings instead.
  • Loading branch information
charliermarsh committed Sep 14, 2024
1 parent 083ec2f commit 3060fd2
Show file tree
Hide file tree
Showing 6 changed files with 4 additions and 35 deletions.
8 changes: 4 additions & 4 deletions crates/uv-settings/src/lib.rs
Original file line number Diff line number Diff line change
@@ -1,10 +1,10 @@
use std::ops::Deref;
use std::path::{Path, PathBuf};

use tracing::debug;
use tracing::{debug, warn};

use uv_fs::Simplified;
use uv_warnings::warn_user;
use uv_warnings::owo_colors::OwoColorize;

pub use crate::combine::*;
pub use crate::settings::*;
Expand Down Expand Up @@ -74,7 +74,7 @@ impl FilesystemOptions {
}
Err(Error::PyprojectToml(file, err)) => {
// If we see an invalid `pyproject.toml`, warn but continue.
warn_user!(
warn!(
"Failed to parse `{}` during settings discovery:\n{}",
file.cyan(),
textwrap::indent(&err.to_string(), " ")
Expand Down Expand Up @@ -107,7 +107,7 @@ impl FilesystemOptions {
.and_then(|content| toml::from_str::<PyProjectToml>(&content).ok())
{
if pyproject.tool.is_some_and(|tool| tool.uv.is_some()) {
warn_user!(
warn!(
"Found both a `uv.toml` file and a `[tool.uv]` section in an adjacent `pyproject.toml`. The `[tool.uv]` section will be ignored in favor of the `uv.toml` file."
);
}
Expand Down
7 changes: 0 additions & 7 deletions crates/uv/tests/lock.rs
Original file line number Diff line number Diff line change
Expand Up @@ -12653,13 +12653,6 @@ fn lock_duplicate_sources() -> Result<()> {
----- stdout -----

----- stderr -----
warning: Failed to parse `pyproject.toml` during settings discovery:
TOML parse error at line 9, column 9
|
9 | python-multipart = { url = "https://files.pythonhosted.org/packages/c0/3e/9fbfd74e7f5b54f653f7ca99d44ceb56e718846920162165061c4c22b71a/python_multipart-0.0.8-py3-none-any.whl" }
| ^
duplicate key `python-multipart` in table `tool.uv.sources`

error: Failed to parse: `pyproject.toml`
Caused by: TOML parse error at line 9, column 9
|
Expand Down
9 changes: 0 additions & 9 deletions crates/uv/tests/pip_compile.rs
Original file line number Diff line number Diff line change
Expand Up @@ -3285,15 +3285,6 @@ fn override_dependency_from_workspace_invalid_syntax() -> Result<()> {
----- stdout -----
----- stderr -----
warning: Failed to parse `pyproject.toml` during settings discovery:
TOML parse error at line 9, column 29
|
9 | override-dependencies = [
| ^
no such comparison operator "=", must be one of ~= == != <= >= < > ===
werkzeug=2.3.0
^^^^^^
error: Failed to parse: `pyproject.toml`
Caused by: TOML parse error at line 9, column 29
|
Expand Down
7 changes: 0 additions & 7 deletions crates/uv/tests/pip_install.rs
Original file line number Diff line number Diff line change
Expand Up @@ -92,13 +92,6 @@ fn invalid_pyproject_toml_syntax() -> Result<()> {
----- stdout -----
----- stderr -----
warning: Failed to parse `pyproject.toml` during settings discovery:
TOML parse error at line 1, column 5
|
1 | 123 - 456
| ^
expected `.`, `=`
error: Failed to parse: `pyproject.toml`
Caused by: TOML parse error at line 1, column 5
|
Expand Down
1 change: 0 additions & 1 deletion crates/uv/tests/show_settings.rs
Original file line number Diff line number Diff line change
Expand Up @@ -2892,7 +2892,6 @@ fn resolve_both() -> anyhow::Result<()> {
}
----- stderr -----
warning: Found both a `uv.toml` file and a `[tool.uv]` section in an adjacent `pyproject.toml`. The `[tool.uv]` section will be ignored in favor of the `uv.toml` file.
"###
);

Expand Down
7 changes: 0 additions & 7 deletions crates/uv/tests/venv.rs
Original file line number Diff line number Diff line change
Expand Up @@ -494,13 +494,6 @@ fn create_venv_warns_user_on_requires_python_discovery_error() -> Result<()> {
----- stdout -----
----- stderr -----
warning: Failed to parse `pyproject.toml` during settings discovery:
TOML parse error at line 1, column 9
|
1 | invalid toml
| ^
expected `.`, `=`
warning: Failed to parse: `pyproject.toml`
Using Python 3.12.[X] interpreter at: [PYTHON-3.12]
Creating virtualenv at: .venv
Expand Down

0 comments on commit 3060fd2

Please sign in to comment.