.mailmap: fix broken line with multiple emails#153520
.mailmap: fix broken line with multiple emails#153520rust-bors[bot] merged 1 commit intorust-lang:mainfrom
Conversation
|
r? @jieyouxu rustbot has assigned @jieyouxu. Use Why was this reviewer chosen?The reviewer was selected based on:
|
|
r? lolbinarycat |
|
this file is used by thanks.rust-lang.org to normalize emails, and i have commits using that email in other repositories. so... yeah that's in there for a reason. |
Okay, but it is currently broken - I discovered this while working on the thanks tool, which complains about the line |
Split binarycat's entry into multiple lines, one for each extra email
|
sorry but why was i pinged here? |
Because you had approved the previous patch - I figured you you might want to know that the old version with multiple emails was invalid, sorry if that was wrong |
can you show me the actual error/warning it prints out? |
It says See, e.g., the most recent automatic build at https://github.com/rust-lang/thanks/actions/runs/22787734149/job/66108074440, "Build static website" section line 164 of the output |
|
Alright, seems good now. @bors r+ |
|
@bors rollup |
…uwer Rollup of 6 pull requests Successful merges: - #152535 (std: use `OnceLock` for Xous environment variables) - #152646 (Update `UnsafeUnpin` impls involving extern types.) - #153559 (Inline and simplify some code for saving incremental data to disk) - #151900 (num: Separate public API from internal implementations) - #153520 (.mailmap: fix broken line with multiple emails) - #153573 (rustdoc-json: fix incorrect documentation for VariantKind::Struct) Failed merges: - #153509 (Cleanup unused diagnostic emission methods - part 2)
Rollup merge of #153520 - DanielEScherzer:patch-4, r=lolbinarycat .mailmap: fix broken line with multiple emails Split binarycat's entry into multiple lines, one for each extra email
Split binarycat's entry into multiple lines, one for each extra email
Follow-up to #149739, CC @lolbinarycat @Kivooeo