-
Notifications
You must be signed in to change notification settings - Fork 10.5k
[NFC] Change magic emoji to __ #35734
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
@swift-ci please smoke test |
Ping. I'd still be nice and good if source code and tests are always editable (and analyzable with scripts) without copying and pasting magic tokens. |
You don't have to copy and paste any magic tokens; if you're writing a test, you don't have to follow what the interface printer happens to do, you can use whatever identifier you like. |
This isn't just about writing tests. It's about accessibility and discovery. I'm a command-line developer. Always have been. My most important tool for exploring a code base is But back to writing tests, if I'm trying to update the swift test suite and if I want to be precise, then I need to either copy-and-paste the emoji or use something like Look, I get it. It's fun and healthy to insert some humor into work. But can we please limit the emoji to documentation, commit messages, etc and not code or the output of code? Also, at least on my preferred development platform, there is a difference between being Unicode correct and Unicode savvy. On Linux, I haven't run into any program with Unicode correctness issues. That being said, font support lags behind Apple's platforms, and the X11 world doesn't seem to be in any hurry to have fallback fonts for when the primary font lacks support for emoji. So at least in my terminal, I just see lots of "can't print this character" boxes (which render fine if I use the web browser, but um, thanks but no thanks). |
Hi @jckarter – I assume that you remain unconvinced/unmoved. Shall I just close this? |
I honestly don't care enough to block it. I'm fine with you merging this. |
Build failed |
@swift-ci smoke test macOS |
As I explained via Twitter, emoji (while fun) are annoying to work with because one cannot easily type them.