mirror of https://github.com/rust-lang/cargo
Compare commits
9 Commits
caf4400467
...
3492c51f97
Author | SHA1 | Date |
---|---|---|
Scott Schafer | 3492c51f97 | |
Scott Schafer | 8a7c93ea20 | |
Scott Schafer | 81ec9e629d | |
Scott Schafer | c94e774d27 | |
Scott Schafer | d9a7b2b260 | |
Scott Schafer | 6b5e093918 | |
bors | eee4ea2f5a | |
Scott Schafer | cf197fc499 | |
Scott Schafer | c3b104e11e |
|
@ -343,7 +343,7 @@ impl FromStr for Edition {
|
|||
}
|
||||
}
|
||||
|
||||
#[derive(PartialEq)]
|
||||
#[derive(Debug, PartialEq)]
|
||||
enum Status {
|
||||
Stable,
|
||||
Unstable,
|
||||
|
@ -387,11 +387,11 @@ macro_rules! features {
|
|||
$(
|
||||
$(#[$attr])*
|
||||
#[doc = concat!("\n\n\nSee <https://doc.rust-lang.org/nightly/cargo/", $docs, ">.")]
|
||||
pub fn $feature() -> &'static Feature {
|
||||
pub const fn $feature() -> &'static Feature {
|
||||
fn get(features: &Features) -> bool {
|
||||
stab!($stab) == Status::Stable || features.$feature
|
||||
}
|
||||
static FEAT: Feature = Feature {
|
||||
const FEAT: Feature = Feature {
|
||||
name: stringify!($feature),
|
||||
stability: stab!($stab),
|
||||
version: $version,
|
||||
|
@ -512,9 +512,10 @@ features! {
|
|||
}
|
||||
|
||||
/// Status and metadata for a single unstable feature.
|
||||
#[derive(Debug)]
|
||||
pub struct Feature {
|
||||
/// Feature name. This is valid Rust identifer so no dash only underscore.
|
||||
name: &'static str,
|
||||
pub name: &'static str,
|
||||
stability: Status,
|
||||
/// Version that this feature was stabilized or removed.
|
||||
version: &'static str,
|
||||
|
|
|
@ -24,7 +24,9 @@ use crate::sources::{PathSource, CRATES_IO_INDEX, CRATES_IO_REGISTRY};
|
|||
use crate::util::edit_distance;
|
||||
use crate::util::errors::{CargoResult, ManifestError};
|
||||
use crate::util::interning::InternedString;
|
||||
use crate::util::lints::{check_im_a_teapot, check_implicit_features, unused_dependencies};
|
||||
use crate::util::lints::{
|
||||
check_im_a_teapot, check_implicit_features, unused_dependencies, verify_lints,
|
||||
};
|
||||
use crate::util::toml::{read_manifest, InheritableFields};
|
||||
use crate::util::{
|
||||
context::CargoResolverConfig, context::CargoResolverPrecedence, context::ConfigRelativePath,
|
||||
|
@ -1147,11 +1149,26 @@ impl<'gctx> Workspace<'gctx> {
|
|||
}
|
||||
|
||||
pub fn emit_warnings(&self) -> CargoResult<()> {
|
||||
let ws_lints = self
|
||||
.root_maybe()
|
||||
.workspace_config()
|
||||
.inheritable()
|
||||
.and_then(|i| i.lints().ok())
|
||||
.unwrap_or_default();
|
||||
|
||||
let ws_cargo_lints = ws_lints
|
||||
.get("cargo")
|
||||
.cloned()
|
||||
.unwrap_or_default()
|
||||
.into_iter()
|
||||
.map(|(k, v)| (k.replace('-', "_"), v))
|
||||
.collect();
|
||||
|
||||
for (path, maybe_pkg) in &self.packages.packages {
|
||||
let path = path.join("Cargo.toml");
|
||||
if let MaybePackage::Package(pkg) = maybe_pkg {
|
||||
if self.gctx.cli_unstable().cargo_lints {
|
||||
self.emit_lints(pkg, &path)?
|
||||
self.emit_lints(pkg, &path, &ws_cargo_lints)?
|
||||
}
|
||||
}
|
||||
let warnings = match maybe_pkg {
|
||||
|
@ -1179,22 +1196,12 @@ impl<'gctx> Workspace<'gctx> {
|
|||
Ok(())
|
||||
}
|
||||
|
||||
pub fn emit_lints(&self, pkg: &Package, path: &Path) -> CargoResult<()> {
|
||||
let ws_lints = self
|
||||
.root_maybe()
|
||||
.workspace_config()
|
||||
.inheritable()
|
||||
.and_then(|i| i.lints().ok())
|
||||
.unwrap_or_default();
|
||||
|
||||
let ws_cargo_lints = ws_lints
|
||||
.get("cargo")
|
||||
.cloned()
|
||||
.unwrap_or_default()
|
||||
.into_iter()
|
||||
.map(|(k, v)| (k.replace('-', "_"), v))
|
||||
.collect();
|
||||
|
||||
pub fn emit_lints(
|
||||
&self,
|
||||
pkg: &Package,
|
||||
path: &Path,
|
||||
ws_cargo_lints: &manifest::TomlToolLints,
|
||||
) -> CargoResult<()> {
|
||||
let mut error_count = 0;
|
||||
let toml_lints = pkg
|
||||
.manifest()
|
||||
|
@ -1212,11 +1219,41 @@ impl<'gctx> Workspace<'gctx> {
|
|||
.map(|(name, lint)| (name.replace('-', "_"), lint))
|
||||
.collect();
|
||||
|
||||
// We should only be using workspace lints if the `[lints]` table is
|
||||
// present in the manifest, and `workspace` is set to `true`
|
||||
let ws_cargo_lints = pkg
|
||||
.manifest()
|
||||
.resolved_toml()
|
||||
.lints
|
||||
.as_ref()
|
||||
.is_some_and(|l| l.workspace)
|
||||
.then(|| ws_cargo_lints);
|
||||
|
||||
let ws_contents = match self.root_maybe() {
|
||||
MaybePackage::Package(pkg) => pkg.manifest().contents(),
|
||||
MaybePackage::Virtual(v) => v.contents(),
|
||||
};
|
||||
|
||||
let ws_document = match self.root_maybe() {
|
||||
MaybePackage::Package(pkg) => pkg.manifest().document(),
|
||||
MaybePackage::Virtual(v) => v.document(),
|
||||
};
|
||||
|
||||
verify_lints(
|
||||
pkg,
|
||||
&path,
|
||||
&normalized_lints,
|
||||
ws_cargo_lints,
|
||||
ws_contents,
|
||||
ws_document,
|
||||
self.root_manifest(),
|
||||
self.gctx,
|
||||
)?;
|
||||
check_im_a_teapot(
|
||||
pkg,
|
||||
&path,
|
||||
&normalized_lints,
|
||||
&ws_cargo_lints,
|
||||
ws_cargo_lints,
|
||||
&mut error_count,
|
||||
self.gctx,
|
||||
)?;
|
||||
|
@ -1224,7 +1261,7 @@ impl<'gctx> Workspace<'gctx> {
|
|||
pkg,
|
||||
&path,
|
||||
&normalized_lints,
|
||||
&ws_cargo_lints,
|
||||
ws_cargo_lints,
|
||||
&mut error_count,
|
||||
self.gctx,
|
||||
)?;
|
||||
|
@ -1232,7 +1269,7 @@ impl<'gctx> Workspace<'gctx> {
|
|||
pkg,
|
||||
&path,
|
||||
&normalized_lints,
|
||||
&ws_cargo_lints,
|
||||
ws_cargo_lints,
|
||||
&mut error_count,
|
||||
self.gctx,
|
||||
)?;
|
||||
|
|
|
@ -1,6 +1,6 @@
|
|||
use crate::core::dependency::DepKind;
|
||||
use crate::core::FeatureValue::Dep;
|
||||
use crate::core::{Edition, FeatureValue, Package};
|
||||
use crate::core::{Edition, Feature, FeatureValue, Manifest, Package};
|
||||
use crate::util::interning::InternedString;
|
||||
use crate::{CargoResult, GlobalContext};
|
||||
use annotate_snippets::{Level, Renderer, Snippet};
|
||||
|
@ -12,11 +12,202 @@ use std::ops::Range;
|
|||
use std::path::Path;
|
||||
use toml_edit::ImDocument;
|
||||
|
||||
const LINT_GROUPS: &[LintGroup] = &[TEST_DUMMY_UNSTABLE];
|
||||
const LINTS: &[Lint] = &[IM_A_TEAPOT, IMPLICIT_FEATURES, UNUSED_OPTIONAL_DEPENDENCY];
|
||||
|
||||
pub fn verify_lints(
|
||||
pkg: &Package,
|
||||
path: &Path,
|
||||
pkg_lints: &TomlToolLints,
|
||||
ws_lints: Option<&TomlToolLints>,
|
||||
ws_contents: &str,
|
||||
ws_document: &ImDocument<String>,
|
||||
ws_path: &Path,
|
||||
gctx: &GlobalContext,
|
||||
) -> CargoResult<()> {
|
||||
let mut error_count = 0;
|
||||
let manifest = pkg.manifest();
|
||||
let manifest_path = rel_cwd_manifest_path(path, gctx);
|
||||
let ws_path = rel_cwd_manifest_path(ws_path, gctx);
|
||||
|
||||
for lint_name in pkg_lints
|
||||
.keys()
|
||||
.chain(ws_lints.map(|l| l.keys()).unwrap_or_default())
|
||||
{
|
||||
if let Some((name, default_level, edition_lint_opts, feature_gate)) =
|
||||
find_lint_or_group(lint_name)
|
||||
{
|
||||
let (_, reason, _) = level_priority(
|
||||
name,
|
||||
*default_level,
|
||||
*edition_lint_opts,
|
||||
pkg_lints,
|
||||
ws_lints,
|
||||
manifest.edition(),
|
||||
);
|
||||
if let Some(feature_gate) = feature_gate {
|
||||
feature_gated_lint(
|
||||
name,
|
||||
feature_gate,
|
||||
reason,
|
||||
manifest,
|
||||
&manifest_path,
|
||||
ws_contents,
|
||||
ws_document,
|
||||
&ws_path,
|
||||
&mut error_count,
|
||||
gctx,
|
||||
)?;
|
||||
}
|
||||
}
|
||||
}
|
||||
if error_count > 0 {
|
||||
Err(anyhow::anyhow!(
|
||||
"encountered {error_count} errors(s) while verifying lints",
|
||||
))
|
||||
} else {
|
||||
Ok(())
|
||||
}
|
||||
}
|
||||
|
||||
fn find_lint_or_group<'a>(
|
||||
name: &str,
|
||||
) -> Option<(
|
||||
&'static str,
|
||||
&LintLevel,
|
||||
&Option<(Edition, LintLevel)>,
|
||||
&Option<&'static Feature>,
|
||||
)> {
|
||||
if let Some(lint) = LINTS.iter().find(|l| l.name == name) {
|
||||
Some((
|
||||
lint.name,
|
||||
&lint.default_level,
|
||||
&lint.edition_lint_opts,
|
||||
&lint.feature_gate,
|
||||
))
|
||||
} else if let Some(group) = LINT_GROUPS.iter().find(|g| g.name == name) {
|
||||
Some((
|
||||
group.name,
|
||||
&group.default_level,
|
||||
&group.edition_lint_opts,
|
||||
&group.feature_gate,
|
||||
))
|
||||
} else {
|
||||
None
|
||||
}
|
||||
}
|
||||
|
||||
fn feature_gated_lint(
|
||||
lint_name: &str,
|
||||
feature_gate: &Feature,
|
||||
reason: LintLevelReason,
|
||||
manifest: &Manifest,
|
||||
manifest_path: &str,
|
||||
ws_contents: &str,
|
||||
ws_document: &ImDocument<String>,
|
||||
ws_path: &str,
|
||||
error_count: &mut usize,
|
||||
gctx: &GlobalContext,
|
||||
) -> CargoResult<()> {
|
||||
if !manifest.unstable_features().is_enabled(feature_gate) {
|
||||
let dash_name = lint_name.replace("_", "-");
|
||||
let dash_feature_name = feature_gate.name.replace("_", "-");
|
||||
let title = format!("use of unstable lint `{}`", dash_name);
|
||||
let label = format!(
|
||||
"this is behind `{}`, which is not enabled",
|
||||
dash_feature_name
|
||||
);
|
||||
let second_title = format!("`cargo::{}` was inherited", dash_name);
|
||||
let help = format!(
|
||||
"consider adding `cargo-features = [\"{}\"]` to the top of the manifest",
|
||||
dash_feature_name
|
||||
);
|
||||
let message = match reason {
|
||||
LintLevelReason::Package => {
|
||||
let span = get_span(
|
||||
manifest.document(),
|
||||
&["lints", "cargo", dash_name.as_str()],
|
||||
false,
|
||||
)
|
||||
.or(get_span(
|
||||
manifest.document(),
|
||||
&["lints", "cargo", lint_name],
|
||||
false,
|
||||
))
|
||||
.unwrap();
|
||||
Some(
|
||||
Level::Error
|
||||
.title(&title)
|
||||
.snippet(
|
||||
Snippet::source(manifest.contents())
|
||||
.origin(&manifest_path)
|
||||
.annotation(Level::Error.span(span).label(&label))
|
||||
.fold(true),
|
||||
)
|
||||
.footer(Level::Help.title(&help)),
|
||||
)
|
||||
}
|
||||
LintLevelReason::Workspace => {
|
||||
let lint_span = get_span(
|
||||
ws_document,
|
||||
&["workspace", "lints", "cargo", dash_name.as_str()],
|
||||
false,
|
||||
)
|
||||
.or(get_span(
|
||||
ws_document,
|
||||
&["workspace", "lints", "cargo", lint_name],
|
||||
false,
|
||||
))
|
||||
.unwrap();
|
||||
let inherit_span_key =
|
||||
get_span(manifest.document(), &["lints", "workspace"], false).unwrap();
|
||||
let inherit_span_value =
|
||||
get_span(manifest.document(), &["lints", "workspace"], true).unwrap();
|
||||
Some(
|
||||
Level::Error
|
||||
.title(&title)
|
||||
.snippet(
|
||||
Snippet::source(ws_contents)
|
||||
.origin(&ws_path)
|
||||
.annotation(Level::Error.span(lint_span).label(&label))
|
||||
.fold(true),
|
||||
)
|
||||
.footer(
|
||||
Level::Note.title(&second_title).snippet(
|
||||
Snippet::source(manifest.contents())
|
||||
.origin(&manifest_path)
|
||||
.annotation(
|
||||
Level::Note
|
||||
.span(inherit_span_key.start..inherit_span_value.end),
|
||||
)
|
||||
.fold(true),
|
||||
),
|
||||
)
|
||||
.footer(Level::Help.title(&help)),
|
||||
)
|
||||
}
|
||||
_ => None,
|
||||
};
|
||||
|
||||
if let Some(message) = message {
|
||||
*error_count += 1;
|
||||
let renderer = Renderer::styled().term_width(
|
||||
gctx.shell()
|
||||
.err_width()
|
||||
.diagnostic_terminal_width()
|
||||
.unwrap_or(annotate_snippets::renderer::DEFAULT_TERM_WIDTH),
|
||||
);
|
||||
writeln!(gctx.shell().err(), "{}", renderer.render(message))?;
|
||||
}
|
||||
}
|
||||
Ok(())
|
||||
}
|
||||
|
||||
fn get_span(document: &ImDocument<String>, path: &[&str], get_value: bool) -> Option<Range<usize>> {
|
||||
let mut table = document.as_item().as_table_like().unwrap();
|
||||
let mut table = document.as_item().as_table_like()?;
|
||||
let mut iter = path.into_iter().peekable();
|
||||
while let Some(key) = iter.next() {
|
||||
let (key, item) = table.get_key_value(key).unwrap();
|
||||
let (key, item) = table.get_key_value(key)?;
|
||||
if iter.peek().is_none() {
|
||||
return if get_value {
|
||||
item.span()
|
||||
|
@ -66,6 +257,7 @@ pub struct LintGroup {
|
|||
pub default_level: LintLevel,
|
||||
pub desc: &'static str,
|
||||
pub edition_lint_opts: Option<(Edition, LintLevel)>,
|
||||
pub feature_gate: Option<&'static Feature>,
|
||||
}
|
||||
|
||||
const TEST_DUMMY_UNSTABLE: LintGroup = LintGroup {
|
||||
|
@ -73,6 +265,7 @@ const TEST_DUMMY_UNSTABLE: LintGroup = LintGroup {
|
|||
desc: "test_dummy_unstable is meant to only be used in tests",
|
||||
default_level: LintLevel::Allow,
|
||||
edition_lint_opts: None,
|
||||
feature_gate: Some(Feature::test_dummy_unstable()),
|
||||
};
|
||||
|
||||
#[derive(Copy, Clone, Debug)]
|
||||
|
@ -82,13 +275,14 @@ pub struct Lint {
|
|||
pub groups: &'static [LintGroup],
|
||||
pub default_level: LintLevel,
|
||||
pub edition_lint_opts: Option<(Edition, LintLevel)>,
|
||||
pub feature_gate: Option<&'static Feature>,
|
||||
}
|
||||
|
||||
impl Lint {
|
||||
pub fn level(
|
||||
&self,
|
||||
pkg_lints: &TomlToolLints,
|
||||
ws_lints: &TomlToolLints,
|
||||
ws_lints: Option<&TomlToolLints>,
|
||||
edition: Edition,
|
||||
) -> (LintLevel, LintLevelReason) {
|
||||
self.groups
|
||||
|
@ -164,7 +358,7 @@ impl From<TomlLintLevel> for LintLevel {
|
|||
}
|
||||
}
|
||||
|
||||
#[derive(Copy, Clone, Debug)]
|
||||
#[derive(Copy, Clone, Debug, PartialEq, Eq)]
|
||||
pub enum LintLevelReason {
|
||||
Default,
|
||||
Edition(Edition),
|
||||
|
@ -188,7 +382,7 @@ fn level_priority(
|
|||
default_level: LintLevel,
|
||||
edition_lint_opts: Option<(Edition, LintLevel)>,
|
||||
pkg_lints: &TomlToolLints,
|
||||
ws_lints: &TomlToolLints,
|
||||
ws_lints: Option<&TomlToolLints>,
|
||||
edition: Edition,
|
||||
) -> (LintLevel, LintLevelReason, i8) {
|
||||
let (unspecified_level, reason) = if let Some(level) = edition_lint_opts
|
||||
|
@ -211,7 +405,7 @@ fn level_priority(
|
|||
LintLevelReason::Package,
|
||||
defined_level.priority(),
|
||||
)
|
||||
} else if let Some(defined_level) = ws_lints.get(name) {
|
||||
} else if let Some(defined_level) = ws_lints.and_then(|l| l.get(name)) {
|
||||
(
|
||||
defined_level.level().into(),
|
||||
LintLevelReason::Workspace,
|
||||
|
@ -228,18 +422,21 @@ const IM_A_TEAPOT: Lint = Lint {
|
|||
groups: &[TEST_DUMMY_UNSTABLE],
|
||||
default_level: LintLevel::Allow,
|
||||
edition_lint_opts: None,
|
||||
feature_gate: Some(Feature::test_dummy_unstable()),
|
||||
};
|
||||
|
||||
pub fn check_im_a_teapot(
|
||||
pkg: &Package,
|
||||
path: &Path,
|
||||
pkg_lints: &TomlToolLints,
|
||||
ws_lints: &TomlToolLints,
|
||||
ws_lints: Option<&TomlToolLints>,
|
||||
error_count: &mut usize,
|
||||
gctx: &GlobalContext,
|
||||
) -> CargoResult<()> {
|
||||
let manifest = pkg.manifest();
|
||||
let manifest_path = rel_cwd_manifest_path(path, gctx);
|
||||
let (lint_level, reason) = IM_A_TEAPOT.level(pkg_lints, ws_lints, manifest.edition());
|
||||
|
||||
if lint_level == LintLevel::Allow {
|
||||
return Ok(());
|
||||
}
|
||||
|
@ -253,7 +450,6 @@ pub fn check_im_a_teapot(
|
|||
*error_count += 1;
|
||||
}
|
||||
let level = lint_level.to_diagnostic_level();
|
||||
let manifest_path = rel_cwd_manifest_path(path, gctx);
|
||||
let emitted_reason = format!(
|
||||
"`cargo::{}` is set to `{lint_level}` {reason}",
|
||||
IM_A_TEAPOT.name
|
||||
|
@ -300,13 +496,14 @@ const IMPLICIT_FEATURES: Lint = Lint {
|
|||
groups: &[],
|
||||
default_level: LintLevel::Allow,
|
||||
edition_lint_opts: None,
|
||||
feature_gate: None,
|
||||
};
|
||||
|
||||
pub fn check_implicit_features(
|
||||
pkg: &Package,
|
||||
path: &Path,
|
||||
pkg_lints: &TomlToolLints,
|
||||
ws_lints: &TomlToolLints,
|
||||
ws_lints: Option<&TomlToolLints>,
|
||||
error_count: &mut usize,
|
||||
gctx: &GlobalContext,
|
||||
) -> CargoResult<()> {
|
||||
|
@ -384,13 +581,14 @@ const UNUSED_OPTIONAL_DEPENDENCY: Lint = Lint {
|
|||
groups: &[],
|
||||
default_level: LintLevel::Warn,
|
||||
edition_lint_opts: None,
|
||||
feature_gate: None,
|
||||
};
|
||||
|
||||
pub fn unused_dependencies(
|
||||
pkg: &Package,
|
||||
path: &Path,
|
||||
pkg_lints: &TomlToolLints,
|
||||
ws_lints: &TomlToolLints,
|
||||
ws_lints: Option<&TomlToolLints>,
|
||||
error_count: &mut usize,
|
||||
gctx: &GlobalContext,
|
||||
) -> CargoResult<()> {
|
||||
|
|
|
@ -982,3 +982,147 @@ error: `im_a_teapot` is specified
|
|||
)
|
||||
.run();
|
||||
}
|
||||
|
||||
#[cargo_test]
|
||||
fn dont_always_inherit_workspace_lints() {
|
||||
let p = project()
|
||||
.file(
|
||||
"Cargo.toml",
|
||||
r#"
|
||||
[workspace]
|
||||
members = ["foo"]
|
||||
|
||||
[workspace.lints.cargo]
|
||||
im-a-teapot = "warn"
|
||||
"#,
|
||||
)
|
||||
.file(
|
||||
"foo/Cargo.toml",
|
||||
r#"
|
||||
cargo-features = ["test-dummy-unstable"]
|
||||
|
||||
[package]
|
||||
name = "foo"
|
||||
version = "0.0.1"
|
||||
edition = "2015"
|
||||
authors = []
|
||||
im-a-teapot = true
|
||||
"#,
|
||||
)
|
||||
.file("foo/src/lib.rs", "")
|
||||
.build();
|
||||
|
||||
p.cargo("check -Zcargo-lints")
|
||||
.masquerade_as_nightly_cargo(&["cargo-lints"])
|
||||
.with_stderr(
|
||||
"\
|
||||
[CHECKING] foo v0.0.1 ([CWD]/foo)
|
||||
[FINISHED] [..]
|
||||
",
|
||||
)
|
||||
.run();
|
||||
}
|
||||
|
||||
#[cargo_test]
|
||||
fn check_feature_gated() {
|
||||
let p = project()
|
||||
.file(
|
||||
"Cargo.toml",
|
||||
r#"
|
||||
[package]
|
||||
name = "foo"
|
||||
version = "0.0.1"
|
||||
edition = "2015"
|
||||
authors = []
|
||||
|
||||
[lints.cargo]
|
||||
im-a-teapot = "warn"
|
||||
"#,
|
||||
)
|
||||
.file("src/lib.rs", "")
|
||||
.build();
|
||||
|
||||
p.cargo("check -Zcargo-lints")
|
||||
.masquerade_as_nightly_cargo(&["cargo-lints"])
|
||||
.with_status(101)
|
||||
.with_stderr(
|
||||
"\
|
||||
error: use of unstable lint `im-a-teapot`
|
||||
--> Cargo.toml:9:1
|
||||
|
|
||||
9 | im-a-teapot = \"warn\"
|
||||
| ^^^^^^^^^^^ this is behind `test-dummy-unstable`, which is not enabled
|
||||
|
|
||||
= help: consider adding `cargo-features = [\"test-dummy-unstable\"]` to the top of the manifest
|
||||
error: encountered 1 errors(s) while verifying lints
|
||||
",
|
||||
)
|
||||
.run();
|
||||
}
|
||||
|
||||
#[cargo_test]
|
||||
fn check_feature_gated_workspace() {
|
||||
let p = project()
|
||||
.file(
|
||||
"Cargo.toml",
|
||||
r#"
|
||||
[workspace]
|
||||
members = ["foo"]
|
||||
|
||||
[workspace.lints.cargo]
|
||||
im-a-teapot = { level = "warn", priority = 10 }
|
||||
test-dummy-unstable = { level = "forbid", priority = -1 }
|
||||
"#,
|
||||
)
|
||||
.file(
|
||||
"foo/Cargo.toml",
|
||||
r#"
|
||||
[package]
|
||||
name = "foo"
|
||||
version = "0.0.1"
|
||||
edition = "2015"
|
||||
authors = []
|
||||
|
||||
[lints]
|
||||
workspace = true
|
||||
"#,
|
||||
)
|
||||
.file("foo/src/lib.rs", "")
|
||||
.build();
|
||||
|
||||
p.cargo("check -Zcargo-lints")
|
||||
.masquerade_as_nightly_cargo(&["cargo-lints"])
|
||||
.with_status(101)
|
||||
.with_stderr(
|
||||
"\
|
||||
error: use of unstable lint `im-a-teapot`
|
||||
--> Cargo.toml:6:1
|
||||
|
|
||||
6 | im-a-teapot = { level = \"warn\", priority = 10 }
|
||||
| ^^^^^^^^^^^ this is behind `test-dummy-unstable`, which is not enabled
|
||||
|
|
||||
note: `cargo::im-a-teapot` was inherited
|
||||
--> foo/Cargo.toml:9:1
|
||||
|
|
||||
9 | workspace = true
|
||||
| ----------------
|
||||
|
|
||||
= help: consider adding `cargo-features = [\"test-dummy-unstable\"]` to the top of the manifest
|
||||
error: use of unstable lint `test-dummy-unstable`
|
||||
--> Cargo.toml:7:1
|
||||
|
|
||||
7 | test-dummy-unstable = { level = \"forbid\", priority = -1 }
|
||||
| ^^^^^^^^^^^^^^^^^^^ this is behind `test-dummy-unstable`, which is not enabled
|
||||
|
|
||||
note: `cargo::test-dummy-unstable` was inherited
|
||||
--> foo/Cargo.toml:9:1
|
||||
|
|
||||
9 | workspace = true
|
||||
| ----------------
|
||||
|
|
||||
= help: consider adding `cargo-features = [\"test-dummy-unstable\"]` to the top of the manifest
|
||||
error: encountered 2 errors(s) while verifying lints
|
||||
",
|
||||
)
|
||||
.run();
|
||||
}
|
||||
|
|
Loading…
Reference in New Issue