Skip to content

refactor: rename _ex() -> _ext() - #8548

Merged
WofWca merged 1 commit into
mainfrom
wofwca/rename-ex-ext
Aug 14, 2026
Merged

refactor: rename _ex() -> _ext()#8548
WofWca merged 1 commit into
mainfrom
wofwca/rename-ex-ext

Conversation

@WofWca

@WofWca WofWca commented Aug 6, 2026

Copy link
Copy Markdown
Member

"ex" means "used to be, but no longer is".
"ext" means "extended", which is the intended meaning.
"Ext" is more common not only in the Rust ecosystem but in general.
We already had several people (myself included) asking
what "ex" is supposed to mean.

To reproduce this commit, search and replace _ex(?!\w) -> _ext.
But don't change list_transports_ex because it's public API.

"ex" means "used to be, but no longer is".
"ext" means "extended", which is the intended meaning.
"Ext" is more common not only in the Rust ecosystem but in general.
We already had several people (myself included) asking
what "ex" is supposed to mean.

To reproduce this commit, search and replace `_ex(?!\w)` -> `_ext`.
But don't change `list_transports_ex` because it's public API.
@WofWca
WofWca force-pushed the wofwca/rename-ex-ext branch from b351b64 to 810c031 Compare August 14, 2026 10:28
@WofWca

WofWca commented Aug 14, 2026

Copy link
Copy Markdown
Member Author

Rebased. Had to do a rename in one newly introduced place, here

core/src/chat.rs

Line 4023 in 810c031

resend_msgs_ext(context, &msgs, contact.fingerprint())

@WofWca

WofWca commented Aug 14, 2026

Copy link
Copy Markdown
Member Author

Another team member saw this MR, didn't object to it.
Merging.

@WofWca
WofWca enabled auto-merge (rebase) August 14, 2026 10:31
@WofWca
WofWca merged commit fbbe56c into main Aug 14, 2026
30 checks passed
@WofWca
WofWca deleted the wofwca/rename-ex-ext branch August 14, 2026 10:40
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