这是indexloc提供的服务,不要输入任何密码
Skip to content

Conversation

@martinfrances107
Copy link
Contributor

All changes are of the form :-

- _ => Err(std::io::Error::new(ErrorKind::Other, "only .ply4d and .gc4d supported")),
+ _ => Err(std::io::Error::other("only .ply4d and .gc4d supported")),

All changes are of the form :-

```
- _ => Err(std::io::Error::new(ErrorKind::Other, "only .ply4d and .gc4d supported")),
+ _ => Err(std::io::Error::other("only .ply4d and .gc4d supported")),
```
@martinfrances107
Copy link
Contributor Author

ok Clippy you are drunk... running clippy locally I get directed one way .. running on github .. I am told to back it out...

@mosure
Copy link
Owner

mosure commented Apr 6, 2025

dang it clippy! I'll see what's up with the ci, maybe some nightly vs. stable difference.

@martinfrances107
Copy link
Contributor Author

I would like to reopen this ... until we get consitency.
I tend to keep up setup upto date .. but that is just a naive way of saying "works on my machine"

cargo version -v

cargo 1.86.0 (adf9b6ad1 2025-02-28)
release: 1.86.0
commit-hash: adf9b6ad14cfa10ff680d5806741a144f7163698
commit-date: 2025-02-28
host: x86_64-unknown-linux-gnu
libgit2: 1.9.0 (sys:0.20.0 vendored)
libcurl: 8.12.0-DEV (sys:0.4.79+curl-8.12.0 vendored ssl:OpenSSL/1.1.1w)
ssl: OpenSSL 1.1.1w  11 Sep 2023
os: Ubuntu 24.4.0 (noble) [64-bit]

rustc --version
rustc 1.86.0 (05f9846f8 2025-03-31)

@martinfrances107
Copy link
Contributor Author

So it look like a sticky cache issue?

I generated 3 single issue PRs fixing 3 unrelated issues...

#184 #185 #186

Now all those test runs report a clippy failure which will be fixed by this PR.

"Something got clogged in the pipes."

@mosure
Copy link
Owner

mosure commented Apr 7, 2025

it looks like there is another one here: src\io\scene.rs:179:22

@martinfrances107
Copy link
Contributor Author

Thanks, fixed

@mosure mosure merged commit 1ee3eb9 into mosure:main Apr 7, 2025
17 checks passed
@martinfrances107 martinfrances107 deleted the error_new branch April 8, 2025 05:31
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