-
Notifications
You must be signed in to change notification settings - Fork 35
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
Specifying language version #196
Comments
That sounds like an ideal solution to me 😄 So would the Unibeautify config look something like the following?
What about language dialects or options?
|
Something like that. Maybe using I also want the language versions to be discrete / enum. So it is not simply numbers for some languages. I'd want Unibeautify to have TypeScript definitions saying We should think a little more about what these version names should be. |
Might be better, yes. The distinction between language dialects and language versions is giving me trouble. Probably they should be combined into the same option, but what to call it? And then language options on top of that (i.e. particular features that the user can pick and choose from, on top of the base language). Would
Good idea. Agreed.
In general, I'd err on the side of extra verbosity. In this case, e.g. XHTML is basically the same language as HTML, but should still be differentiated. So the HTML version would cover both XHTML and classic (SGML-based) HTML.
Do our other enum options prefer upper or lowercase for their values?
We can actually make do with very few versions of C if we handle most differences as language features (instead of them being different versions of the base language).
Hmm. I guess most people don't actually know what version of a language they are using 😄 So it would be easier if there's a default. Also I think the majority of external formatters don't currently have a language version option. Then again, explicit is often better than implicit. |
The other enums seem to be using
|
@lassik This looks great! 🎉 |
Go 1.12 release notes:
|
Quoting @Glavin001 from here:
The text was updated successfully, but these errors were encountered: