CI: select probe for on-target test

rust-embedded / cortex-m

Low level access to Cortex-M processors

Apache License 2.0

835 stars 152 forks source link

Closed newAM closed 5 months ago

adamgreig commented 5 months ago

Ugh, it looks like a syn minor version bump increased their MSRV to 1.61, beyond our current 1.60...

newAM commented 5 months ago

I attempted to pin syn, with something similar to:

[dependencies.syn]
version = ">=2.0.0,<2.0.53"
optional = true

[features]
serde = ["dep:serde", "dep:syn"]

However our MSRV doesn't support dep: either.

I could pin our lockfile in CI, but that doesn't solve the problem for downstream consumers.

I think the easiest way out for now is an MSRV bump?

adamgreig commented 5 months ago

Yep, I think go ahead with an MSRV bump in this PR.