mirror of
https://github.com/martinvonz/jj.git
synced 2025-05-05 23:42:50 +00:00
448f1fd630
This is slightly less pretty than making an Email type,
but I think is much easier.
I would use instead of the last argument of:
```
concat(
label("email", signature.username()),
"@",
label("email",
signature.email().remove_prefix(
signature.username()
).remove_prefix("@")
)
)
```
(Aside for more motivation: I initially had a bug above by using
`.remove_prefix(signature.username() ++ "@")`. This is wrong since an
email might not contain an `@`, as I discovered while looking at the
code. The bug is now fixed, but `signature.email_domain()` would avoid
the problem)