Join GitHub today
GitHub is home to over 20 million developers working together to host and review code, manage projects, and build software together.
chore(Conventions): naming standards #196
Conversation
NathanWalker
commented on the diff
Jan 4, 2016
| @@ -27,7 +27,7 @@ import {XLarge} from './directives/x-large'; | ||
| // Every Angular template is first compiled by the browser before Angular runs it's compiler | ||
| template: require('./home.html') | ||
| }) | ||
| -export class Home { | ||
| +export class HomeCmp implements OnInit { |
NathanWalker
Contributor
|
gdi2290
commented on the diff
Jan 4, 2016
| ]) | ||
| -export class App { | ||
| +export class AppCmp { |
NathanWalker
Contributor
|
|
I have my own reasons for the current structure that I will be covering in a blog post in the future that may conflict with the current naming conventions |
gdi2290
commented on the diff
Jan 4, 2016
|
@gdi2290 Absolutely! Don't merge (as mentioned)! However, maybe we can keep the PR open just to see if others have thoughts that may help form the standards or reach consensus? Idk, I wish I didn't have mixed feelings about the going conventions/standards, but it seems they are amiss on conveniences and simple shorthand. |
|
I wish there was a convert to issue button. I'll include this in the FAQ |
NathanWalker commentedJan 4, 2016
closes AngularClass#185
I'm not saying merge this actually, however this is how the naming conventions would play out here.
I've had mixed feelings about them for awhile but in general, like them.
I've just never been a big fan of long filenames which is why I'm not in love with
whatever.component.tsvs.whatever.ts, however it does make things clear.This becomes more idiomatic when you consider most conventions suggest placing files inside a folder called
componentssuch as shown here in this PR.So when opening files and you see:
app/components/home/home.component.tsvs.
app/home/home.tsFirst, for large projects with more than 2-3 nested hierarchies, you would need a screen or IDE at least wide enough to show the long path and filename when you have multiple tabs open to discern the difference between 2 of the same named files in different folders, ie.:
app/component/home/home.component.tsvs.
app/component/users/profile/home.component.ts^ just an example but it can definitely happen ;)
But in the end, I digress. Whatever the community believes is universally great.
If this is the way to go, then by all means, merge👍