Skip to content
This repository has been archived by the owner on Nov 14, 2023. It is now read-only.

Fix lints #15

Merged
merged 2 commits into from
Jan 22, 2016
Merged

Fix lints #15

merged 2 commits into from
Jan 22, 2016

Conversation

yberreby
Copy link
Collaborator

In 0c223c6, you added new #![deny(..)] lints, without changing the code accordingly. As a result, doapi-rs fails to build because of missing copy and debug implementations, missing docs and "unnecessary qualifications" that are actually useful — that is, the code fails to compile without them.

This PR removes the three problematic lints, fixes a warning (the append feature is now stable) and runs rustfmt (I couldn't easily do it in a separate commit as I have rust.vim configured to run it automatically on save).

@kbknapp
Copy link
Owner

kbknapp commented Jan 22, 2016

👍 Thanks!
@homu r+

@kbknapp
Copy link
Owner

kbknapp commented Jan 22, 2016

Ah guess I'm not using @homu on this repo...sad face

Merged.

kbknapp added a commit that referenced this pull request Jan 22, 2016
@kbknapp kbknapp merged commit b4331c8 into kbknapp:master Jan 22, 2016
@yberreby
Copy link
Collaborator Author

Would you mind pushing a new version to crates.io? It looks like to forgot to do it last time you updated the dependencies. As a result, docli-rs too fails to build on nightly because its doapi-rs crates.io dependency uses serde_macros 0.5.3 instead of 0.6.x as specified in the .toml.

❯ cargo build
 Downloading libc v0.2.5
   Compiling libc v0.2.5
   Compiling serde_macros v0.5.3
/Users/yberreby/.multirust/toolchains/nightly/cargo/registry/src/github.com-88ac128001ac3a9a/serde_macros-0.5.3/src/lib.rs:8:35: 8:58 error: failed to resolve. Could not find `plugin` in `rustc` [E0433]
/Users/yberreby/.multirust/toolchains/nightly/cargo/registry/src/github.com-88ac128001ac3a9a/serde_macros-0.5.3/src/lib.rs:8 pub fn plugin_registrar(reg: &mut rustc::plugin::Registry) {
                                                                                                                                                               ^~~~~~~~~~~~~~~~~~~~~~~
/Users/yberreby/.multirust/toolchains/nightly/cargo/registry/src/github.com-88ac128001ac3a9a/serde_macros-0.5.3/src/lib.rs:8:35: 8:58 help: run `rustc --explain E0433` to see a detailed explanation
/Users/yberreby/.multirust/toolchains/nightly/cargo/registry/src/github.com-88ac128001ac3a9a/serde_macros-0.5.3/src/lib.rs:8:35: 8:58 error: use of undeclared type name `rustc::plugin::Registry` [E0412]
/Users/yberreby/.multirust/toolchains/nightly/cargo/registry/src/github.com-88ac128001ac3a9a/serde_macros-0.5.3/src/lib.rs:8 pub fn plugin_registrar(reg: &mut rustc::plugin::Registry) {
                                                                                                                                                               ^~~~~~~~~~~~~~~~~~~~~~~
/Users/yberreby/.multirust/toolchains/nightly/cargo/registry/src/github.com-88ac128001ac3a9a/serde_macros-0.5.3/src/lib.rs:8:35: 8:58 help: run `rustc --explain E0412` to see a detailed explanation
error: aborting due to 2 previous errors
Build failed, waiting for other jobs to finish...
Could not compile `serde_macros`.
$ rustc --version
rustc 1.7.0-nightly (7dce32e65 2016-01-20)

@kbknapp
Copy link
Owner

kbknapp commented Jan 22, 2016

Done :) v0.1.2 is out on crates.io

Bare in mind though, I haven't messed with this crate or docli in quite some time - and they were built when I was still very new to Rust so there may be some horrendous architecture in there 😜 One of these days I'd love to go through and clean up some of this!

@yberreby
Copy link
Collaborator Author

Don't worry, I noticed it wasn't touched in a while, but I'm thankful you took the time to write it. I'm considering using these two crates for a project of mine.

By the way, I've just made a PR to docli: kbknapp/docli-rs#16

May as well fix both of them, eh? :)

@yberreby yberreby deleted the fix-lints branch March 5, 2016 21:24
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants