diff --git a/.github/workflows/main-checks.yml b/.github/workflows/main-checks.yml index 66fc8b43d71..bcf986342e3 100644 --- a/.github/workflows/main-checks.yml +++ b/.github/workflows/main-checks.yml @@ -216,7 +216,7 @@ jobs: - name: Run tests uses: actions-rs/cargo@v1 env: - RUSTFLAGS: --cfg nightly_yew + RUSTFLAGS: --cfg nightly_yew --cfg yew_lints with: command: test - args: -p yew-macro test_html_lints --features lints + args: -p yew-macro test_html_lints diff --git a/packages/yew-macro/Cargo.toml b/packages/yew-macro/Cargo.toml index 51d3adccfe2..d9eef17b6a3 100644 --- a/packages/yew-macro/Cargo.toml +++ b/packages/yew-macro/Cargo.toml @@ -29,8 +29,3 @@ prettyplease = "0.1.1" rustversion = "1" trybuild = "1" yew = { path = "../yew" } - -[build-dependencies] - -[features] -lints = [] diff --git a/packages/yew-macro/src/html_tree/lint/mod.rs b/packages/yew-macro/src/html_tree/lint/mod.rs index 77fd51d96a3..f925f3a15ef 100644 --- a/packages/yew-macro/src/html_tree/lint/mod.rs +++ b/packages/yew-macro/src/html_tree/lint/mod.rs @@ -26,6 +26,9 @@ pub fn lint(tree: &HtmlTree) where L: Lint, { + #[cfg(not(yew_lints))] + let _ = tree; + #[cfg(yew_lints)] match tree { HtmlTree::List(list) => { for child in &list.children.0 { diff --git a/packages/yew-macro/tests/html_lints/fail.stderr b/packages/yew-macro/tests/html_lints/fail.stderr index 54698329b1d..64551afcdc0 100644 --- a/packages/yew-macro/tests/html_lints/fail.stderr +++ b/packages/yew-macro/tests/html_lints/fail.stderr @@ -1,27 +1,3 @@ -warning: All `` elements should have a `href` attribute. This makes it possible for assistive technologies to correctly interpret what your links point to. https://developer.mozilla.org/en-US/docs/Learn/Accessibility/HTML#more_on_links - --> tests/html_lints/fail.rs:5:10 - | -5 | { "I don't have a href attribute" } - | ^ - -warning: '#' is not a suitable value for the `href` attribute. Without a meaningful attribute assistive technologies will struggle to understand your webpage. https://developer.mozilla.org/en-US/docs/Learn/Accessibility/HTML#onclick_events - --> tests/html_lints/fail.rs:8:17 - | -8 | { "I have a malformed href attribute" } - | ^^^ - -warning: 'javascript:void(0)' is not a suitable value for the `href` attribute. Without a meaningful attribute assistive technologies will struggle to understand your webpage. https://developer.mozilla.org/en-US/docs/Learn/Accessibility/HTML#onclick_events - --> tests/html_lints/fail.rs:11:17 - | -11 | { "I have a malformed href attribute" } - | ^^^^^^^^^^^^^^^^^^^^ - -warning: All `` tags should have an `alt` attribute which provides a human-readable description - --> tests/html_lints/fail.rs:14:10 - | -14 | - | ^^^ - warning: The tag 'tExTAreA' is not matching its normalized form 'textarea'. If you want to keep this form, change this to a dynamic tag `@{"tExTAreA"}`. --> tests/html_lints/fail.rs:17:10 | diff --git a/packages/yew-macro/tests/html_lints_test.rs b/packages/yew-macro/tests/html_lints_test.rs index af71e2d826f..35ba0496c6d 100644 --- a/packages/yew-macro/tests/html_lints_test.rs +++ b/packages/yew-macro/tests/html_lints_test.rs @@ -1,5 +1,5 @@ #[allow(dead_code)] -#[cfg(feature = "lints")] +#[cfg(yew_lints)] #[rustversion::attr(nightly, test)] fn test_html_lints() { let t = trybuild::TestCases::new();