CI: Fix global symbol and lint checks

[JurajSadel]

closes #5571

[MabezDev]

Hmm, the clic module isn't already gated on rt 🤔 ? Are there other parts of the interrupt module we're not gating behind rt properly? Why doesn't this fail on normal CI too 🤔.

[JurajSadel]

Why doesn't this fail on normal CI too 🤔.

The failing command builds esp-hal with --no-default-features --features esp32p4, so rt is disabled

[MabezDev]

Why doesn't this fail on normal CI too 🤔.

The failing command builds esp-hal with --no-default-features --features esp32p4, so rt is disabled

I don't understand what this means? We run the same suite on nightly as we do normally, right? So why doesn't this fail there?

[JurajSadel]

Why doesn't this fail on normal CI too 🤔.

The failing command builds esp-hal with --no-default-features --features esp32p4, so rt is disabled

I don't understand what this means? We run the same suite on nightly as we do normally, right? So why doesn't this fail there?

We are running the global-symbol-check only on nightly, and it builds hal with --no-default-features :

esp-hal/xtask/src/main.rs

Lines 806 to 815 in 72d44d0

	Command::new("cargo")
	.args(&[
	"+esp",
	"build",
	"--no-default-features",
	"--features",
	chip,
	"--target",
	target,
	"-Zbuild-std=core",

[bugadani]

global-symbol-check should probably respect the configurations we list in Cargo.toml, then