Make all the packages consistent (#260)

- **Fix broken links**
- **Delete unnecessary gitignore**
- **Remove doc(html_root_url)**
- **Use workspace authors info**
- **Unify lint config**
- **Use doc_auto_cfg for docs.rs**

Author: pksunkara

.github/workflows/ci.yml

@@ -116,7 +116,7 @@ jobs:
       - uses: actions-rs/cargo@v1
         with:
           command: fmt
-          args: --check
+          args: --all --check
 
   clippy:
     name: Clippy

color-eyre/.github/workflows/ci.yml

@@ -7,23 +7,6 @@ on:
 name: Continuous integration
 
 jobs:
-  check:
-    name: Check
-    runs-on: ubuntu-latest
-    strategy:
-      matrix:
-        rust:
-          - stable
-    steps:
-      - uses: actions/checkout@v1
-      - uses: actions-rs/toolchain@v1
-        with:
-          toolchain: ${{ matrix.rust }}
-          override: true
-      - uses: actions-rs/cargo@v1
-        with:
-          command: check
-
   test-features:
     name: Test Features
     runs-on: ubuntu-latest
@@ -78,58 +61,3 @@ jobs:
       - name: run wasm tests
         run: wasm-pack test --node
         if: ${{ matrix.target == 'wasm32-unknown-unknown' }}
-
-  test-os:
-    name: Test Operating Systems
-    runs-on: ${{ matrix.os }}
-    strategy:
-      matrix:
-        os: [ubuntu-latest, windows-latest, macOS-latest]
-    steps:
-      - uses: actions/checkout@v1
-      - uses: actions-rs/toolchain@v1
-        with:
-          toolchain: stable
-          profile: minimal
-          override: true
-      - uses: actions-rs/cargo@v1
-        with:
-          command: test
-
-  fmt:
-    name: Rustfmt
-    runs-on: ubuntu-latest
-    strategy:
-      matrix:
-        rust:
-          - stable
-    steps:
-      - uses: actions/checkout@v1
-      - uses: actions-rs/toolchain@v1
-        with:
-          toolchain: ${{ matrix.rust }}
-          override: true
-      - run: rustup component add rustfmt
-      - uses: actions-rs/cargo@v1
-        with:
-          command: fmt
-          args: --all -- --check
-
-  clippy:
-    name: Clippy
-    runs-on: ubuntu-latest
-    strategy:
-      matrix:
-        rust:
-          - stable
-    steps:
-      - uses: actions/checkout@v1
-      - uses: actions-rs/toolchain@v1
-        with:
-          toolchain: ${{ matrix.rust }}
-          override: true
-      - run: rustup component add clippy
-      - uses: actions-rs/cargo@v1
-        with:
-          command: clippy
-          args: -- -D warnings

color-eyre/src/config.rs

@@ -546,7 +546,6 @@ impl HookBuilder {
     ///     .unwrap();
     /// ```
     #[cfg(feature = "issue-url")]
-    #[cfg_attr(docsrs, doc(cfg(feature = "issue-url")))]
     pub fn issue_url<S: ToString>(mut self, url: S) -> Self {
         self.issue_url = Some(url.to_string());
         self
@@ -566,7 +565,6 @@ impl HookBuilder {
     ///     .unwrap();
     /// ```
     #[cfg(feature = "issue-url")]
-    #[cfg_attr(docsrs, doc(cfg(feature = "issue-url")))]
     pub fn add_issue_metadata<K, V>(mut self, key: K, value: V) -> Self
     where
         K: Display,
@@ -602,7 +600,6 @@ impl HookBuilder {
     ///     .unwrap();
     ///
     #[cfg(feature = "issue-url")]
-    #[cfg_attr(docsrs, doc(cfg(feature = "issue-url")))]
     pub fn issue_filter<F>(mut self, predicate: F) -> Self
     where
         F: Fn(crate::ErrorKind<'_>) -> bool + Send + Sync + 'static,
@@ -629,7 +626,6 @@ impl HookBuilder {
     ///
     /// This will not disable the location section in a panic message.
     #[cfg(feature = "track-caller")]
-    #[cfg_attr(docsrs, doc(cfg(feature = "track-caller")))]
     pub fn display_location_section(mut self, cond: bool) -> Self {
         self.display_location_section = cond;
         self
@@ -1183,5 +1179,4 @@ pub type FilterCallback = dyn Fn(&mut Vec<&Frame>) + Send + Sync + 'static;
 
 /// Callback for filtering issue url generation in error reports
 #[cfg(feature = "issue-url")]
-#[cfg_attr(docsrs, doc(cfg(feature = "issue-url")))]
 pub type IssueFilterCallback = dyn Fn(crate::ErrorKind<'_>) -> bool + Send + Sync + 'static;

color-eyre/src/handler.rs

@@ -24,7 +24,6 @@ impl Handler {
 
     /// Return a reference to the captured `SpanTrace` type
     #[cfg(feature = "capture-spantrace")]
-    #[cfg_attr(docsrs, doc(cfg(feature = "capture-spantrace")))]
     pub fn span_trace(&self) -> Option<&SpanTrace> {
         self.span_trace.as_ref()
     }

color-eyre/src/lib.rs

@@ -325,20 +325,25 @@
 //! [`eyre::EyreHandler`]: https://docs.rs/eyre/*/eyre/trait.EyreHandler.html
 //! [`backtrace::Backtrace`]: https://docs.rs/backtrace/*/backtrace/struct.Backtrace.html
 //! [`tracing_error::SpanTrace`]: https://docs.rs/tracing-error/*/tracing_error/struct.SpanTrace.html
-//! [`color-spantrace`]: https://github.com/yaahc/color-spantrace
+//! [`color-spantrace`]: https://docs.rs/color-spantrace
 //! [`Section`]: https://docs.rs/color-eyre/*/color_eyre/trait.Section.html
 //! [`eyre::Report`]: https://docs.rs/eyre/*/eyre/struct.Report.html
 //! [`eyre::Result`]: https://docs.rs/eyre/*/eyre/type.Result.html
 //! [`Handler`]: https://docs.rs/color-eyre/*/color_eyre/struct.Handler.html
-//! [`examples/usage.rs`]: https://github.com/yaahc/color-eyre/blob/master/examples/usage.rs
-//! [`examples/custom_filter.rs`]: https://github.com/yaahc/color-eyre/blob/master/examples/custom_filter.rs
-//! [`examples/custom_section.rs`]: https://github.com/yaahc/color-eyre/blob/master/examples/custom_section.rs
-//! [`examples/multiple_errors.rs`]: https://github.com/yaahc/color-eyre/blob/master/examples/multiple_errors.rs
-#![doc(html_root_url = "https://docs.rs/color-eyre/0.6.2")]
-#![cfg_attr(docsrs, feature(doc_cfg))]
+//! [`examples/usage.rs`]: https://github.com/eyre-rs/eyre/blob/master/color-eyre/examples/usage.rs
+//! [`examples/custom_filter.rs`]: https://github.com/eyre-rs/eyre/blob/master/color-eyre/examples/custom_filter.rs
+//! [`examples/custom_section.rs`]: https://github.com/eyre-rs/eyre/blob/master/color-eyre/examples/custom_section.rs
+//! [`examples/multiple_errors.rs`]: https://github.com/eyre-rs/eyre/blob/master/color-eyre/examples/multiple_errors.rs
+#![cfg_attr(
+    nightly,
+    feature(rustdoc_missing_doc_code_examples),
+    warn(rustdoc::missing_doc_code_examples)
+)]
+#![cfg_attr(docsrs, feature(doc_auto_cfg))]
 #![warn(
+    missing_debug_implementations,
     missing_docs,
-    rustdoc::missing_doc_code_examples,
+    unsafe_op_in_unsafe_fn,
     rust_2018_idioms,
     unreachable_pub,
     bad_style,
@@ -349,13 +354,13 @@
     overflowing_literals,
     path_statements,
     patterns_in_fns_without_body,
+    unconditional_recursion,
     unused,
     unused_allocation,
     unused_comparisons,
     unused_parens,
     while_true
 )]
-#![allow(clippy::try_err)]
 
 use std::sync::Arc;
 
@@ -418,7 +423,6 @@ pub struct Handler {
 
 /// The kind of type erased error being reported
 #[cfg(feature = "issue-url")]
-#[cfg_attr(docsrs, doc(cfg(feature = "issue-url")))]
 pub enum ErrorKind<'a> {
     /// A non recoverable error aka `panic!`
     NonRecoverable(&'a dyn std::any::Any),

color-spantrace/.gitignore

@@ -59,15 +59,16 @@
 //!
 //! [`tracing_error::SpanTrace`]: https://docs.rs/tracing-error/*/tracing_error/struct.SpanTrace.html
 //! [`color-backtrace`]: https://github.com/athre0z/color-backtrace
-#![doc(html_root_url = "https://docs.rs/color-spantrace/0.2.1")]
 #![cfg_attr(
     nightly,
     feature(rustdoc_missing_doc_code_examples),
     warn(rustdoc::missing_doc_code_examples)
 )]
+#![cfg_attr(docsrs, feature(doc_auto_cfg))]
 #![warn(
     missing_debug_implementations,
     missing_docs,
+    unsafe_op_in_unsafe_fn,
     rust_2018_idioms,
     unreachable_pub,
     bad_style,
@@ -85,6 +86,7 @@
     unused_parens,
     while_true
 )]
+
 use once_cell::sync::OnceCell;
 use owo_colors::{style, Style};
 use std::env;

color-spantrace/src/lib.rs

@@ -1,11 +1,11 @@
 [package]
 name = "eyre"
 version = "1.0.0"
-authors = ["David Tolnay <dtolnay@gmail.com>", "Jane Lusby <jlusby42@gmail.com>"]
 description = "Flexible concrete Error Reporting type built on std::error::Error with customizable Reports"
 documentation = "https://docs.rs/eyre"
 categories = ["rust-patterns"]
 
+authors = { workspace = true }
 edition = { workspace = true }
 license = { workspace = true }
 repository = { workspace = true }
@@ -39,7 +39,8 @@ pyo3 = { version = "0.25", default-features = false, features = ["auto-initializ
 
 [package.metadata.docs.rs]
 targets = ["x86_64-unknown-linux-gnu"]
-rustdoc-args = ["--cfg", "doc_cfg"]
+all-features = true
+rustdoc-args = ["--cfg", "docsrs"]
 
 [package.metadata.workspaces]
 independent = true

eyre/Cargo.toml

@@ -11,7 +11,7 @@ fn main() {
     println!("cargo:rustc-check-cfg=cfg(eyre_no_fmt_args_capture)");
     println!("cargo:rustc-check-cfg=cfg(backtrace)");
     println!("cargo:rustc-check-cfg=cfg(eyre_no_fmt_arguments_as_str)");
-    println!("cargo:rustc-check-cfg=cfg(doc_cfg)");
+    println!("cargo:rustc-check-cfg=cfg(docsrs)");
     let ac = autocfg::new();
 
     // https://github.com/rust-lang/rust/issues/99301 [nightly]
@@ -49,7 +49,7 @@ const GENERIC_MEMBER_ACCESS_PROBE: &str = r#"
     use std::fmt::{self, Display};
 
     #[derive(Debug)]
-    struct E { 
+    struct E {
         backtrace: MyBacktrace,
     }
 

eyre/build.rs

@@ -328,12 +328,12 @@
 //! [`simple-eyre`]: https://github.com/eyre-rs/simple-eyre
 //! [`color-spantrace`]: https://github.com/eyre-rs/color-spantrace
 //! [`color-backtrace`]: https://github.com/athre0z/color-backtrace
-#![doc(html_root_url = "https://docs.rs/eyre/0.6.11")]
 #![cfg_attr(
     nightly,
     feature(rustdoc_missing_doc_code_examples),
     warn(rustdoc::missing_doc_code_examples)
 )]
+#![cfg_attr(docsrs, feature(doc_auto_cfg))]
 #![warn(
     missing_debug_implementations,
     missing_docs,
@@ -356,7 +356,6 @@
     while_true
 )]
 #![cfg_attr(generic_member_access, feature(error_generic_member_access))]
-#![cfg_attr(doc_cfg, feature(doc_cfg))]
 #![allow(
     clippy::needless_doctest_main,
     clippy::new_ret_no_self,