-
-
Notifications
You must be signed in to change notification settings - Fork 5.5k
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
Capitalization of DevNull
#25786
Comments
Bump. This is a small but backwards-incompatible name consistency change. Should it be done for 1.0? |
Let's hear what triage thinks about this (will happen tomorrow). |
I think we should do this. |
As an aside, the Julia Vim definitions used to highlight |
From triage: there is some significant support for making these all lowercase --- |
But don't we follow the convention that constants use uppercase? |
It turns out neither |
It was pointed out that we don't always: |
I like lowercasing all of these: stdin, stdout, stderr, devnull. @Keno points out that devnull comes from |
Also, |
Triage votes for lowercasing all of these with a dissenting vote from @ararslan. |
I've adjusted #25959 per triage consensus. |
DevNull
is proper case whileSTDOUT
is upper case. Since we generally only use CamelCase for types and modules, this should perhaps be changed to something likeDEVNULL
.(cc @nalimilan)
The text was updated successfully, but these errors were encountered: