Skip to content

Conversation

cpu
Copy link
Member

@cpu cpu commented Feb 21, 2025

ci: limit concurrency, auto-cancel stale jobs

This lifts some configuration we've been using on the main Rustls repo for some time now without issue. The primary advantage is having stale workflow runs automatically cancelled when a branch update is pushed.

Historically for this repo I've done that by hand and it's a pain!

ci: format with latest stable rust

Formatting against MSRV is too restrictive. MSRV is primarily for downstream consumers, and not for developers.

rustfmt: style_edition 2024

Like rustls/rustls#2348, rustls/webpki#323 and rustls/pki-types#74 we want to adopt 2024 edition style without actually taking 2024 edition (and breaking semver).

Unlike some other repos, cargo fix --edition will produce a diff (primarily small changes related to no_mangle being marked unsafe, and unsafe fn functions not having an automatic function-wide unsafe scope). We can't take those until MSRV allows.

cpu added 3 commits February 21, 2025 10:38
This lifts some configuration we've been using on the main Rustls repo
for some time now without issue. The primary advantage is having stale
workflow runs automatically cancelled when a branch update is pushed.

Historically for this repo I've done that by hand and it's a pain!
Formatting against MSRV is too restrictive. MSRV is primarily for
downstream consumers, and not for developers.
@cpu cpu requested a review from ctz February 21, 2025 15:42
@cpu cpu merged commit 48ffc60 into rustls:main Feb 21, 2025
46 checks passed
@cpu cpu deleted the cpu-2024-edition-ish branch February 21, 2025 15:56
yedayak added a commit to yedayak/rustls-ffi that referenced this pull request Jun 24, 2025
yedayak added a commit to yedayak/rustls-ffi that referenced this pull request Jun 24, 2025
djc pushed a commit that referenced this pull request Jun 24, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants