Skip to content

name: transient should prefix with t_ - #64

Merged
apbendi merged 6 commits into
mainfrom
transient
Jul 25, 2026
Merged

name: transient should prefix with t_#64
apbendi merged 6 commits into
mainfrom
transient

Conversation

@thelostone-mc

@thelostone-mc thelostone-mc commented Feb 27, 2026

Copy link
Copy Markdown
Contributor
image

@thelostone-mc
thelostone-mc force-pushed the transient branch 4 times, most recently from 5ae8901 to ea6c274 Compare February 27, 2026 07:39
@thelostone-mc
thelostone-mc marked this pull request as ready for review March 2, 2026 05:51

@apbendi apbendi left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Quick question @garyghayrat

Comment thread src/check/validators/src_names_internal.rs
apbendi added a commit that referenced this pull request Jul 25, 2026
Main's lockfile pinned rustix 0.36.8, which uses rustc_* attributes the
current nightly compiler rejects, breaking the build/test/lint CI jobs.
Lockfile taken verbatim from the transient branch (PR #64), which
already contains these bumps.
apbendi added a commit that referenced this pull request Jul 25, 2026
* chore: fix clippy lints failing CI on latest nightly

The clippy CI job installs the latest nightly clippy on every run, and
newer lint versions now flag existing code on main: a redundant else
after process::exit, redundant borrows in format! args, and an if that
can be collapsed into a match guard.

The variable_names and src_names_internal fixes mirror the identical
changes already made on the transient branch (PR #64) so it rebases
cleanly.

* chore: bump rustix and is-terminal so main builds on latest nightly

Main's lockfile pinned rustix 0.36.8, which uses rustc_* attributes the
current nightly compiler rejects, breaking the build/test/lint CI jobs.
Lockfile taken verbatim from the transient branch (PR #64), which
already contains these bumps.
The offset-stability of the transient keyword stripping depends on the
replacement string having the same length as the keyword; enforce that
at compile time.
@apbendi
apbendi merged commit 735cf46 into main Jul 25, 2026
4 checks passed
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.

3 participants