load all comments

cupcakezealot 30 points a year ago

i would say why would you just not to isAdmin = true but i also worked with someone who did just this so i'll instead just sigh.

also the real crime is the use of javascript tbh

save

path: 0 18801252, hotness: undefined, score: 30, children: 15

Lemminary 23 points a year ago

That's TypeScript. I can tell by the pixels defining a type above.

save

path: 0 18801252 18801371, hotness: undefined, score: 23, children: 14

Maiq 6 points a year ago

Was looking at it and could not figure out why their weren't any semicolon's.

save

path: 0 18801252 18801371 18803066, hotness: undefined, score: 6, children: 13

ScintillatingStruthio 13 points a year ago

Neither Javascript nor Typescript require semicolon, it is entirely a stylistic choice except in very rare circumstances that do not come up in normal code.

save

path: 0 18801252 18801371 18803066 18803965, hotness: undefined, score: 13, children: 12

Maiq 4 points a year ago

That's good to know. Don't know how I didn't know this. Been writing JS since 2000. Always just used them I guess. Ecmascripts look funny to me without them

save

path: 0 18801252 18801371 18803066 18803965 18804200, hotness: undefined, score: 4, children: 4

ScintillatingStruthio 2 points a year ago

Fair enough, I like it better without but I don't have a strong preference and have no issue adapting to whatever the style of the repo is.

I learned about it researching tools to automatically enforce formatting style and came across StandardJS, which eliminates them by default.

save

path: 0 18801252 18801371 18803066 18803965 18804200 18821915, hotness: undefined, score: 2, children: 2

Maiq 1 point a year ago

I can see the benefit of matching style when working with others. I only code for myself and never had to worry about conformity for project consistency.

It is good to learn new things.

I'm sure I have some coding habitats that would annoy others.

save

path: 0 18801252 18801371 18803066 18803965 18804200 18821915 18822350, hotness: undefined, score: 1, children: 1

ScintillatingStruthio 2 points a year ago

Consistent styling helps make the actual meaningful changes easier to spot. Probably also useful for your own commit history when working solo in a repo, but most useful in a team, yeah!

save

path: 0 18801252 18801371 18803066 18803965 18804200 18821915 18822350 18824453, hotness: undefined, score: 2, children: 0

Rules

Rules