3
52
fusio

@lemmy.world

fusio 39 points 3 years ago

That's why PR should be small. It's much better to have multiple PRs than a single big one.

Totally fair to have gigantic PR full of boilerplate code, but generally you can split the boilerplate and your feature in 2 PRs, where only the feature will get a proper review.

All of this obviously depends on the criticality of the system :p

path: 0 727823, hotness: undefined, score: 39, children: 4
fusio 37 points 3 years ago

he can use the money for therapy

path: 0 4652574, hotness: undefined, score: 37, children: 0
fusio 33 points 3 years ago

ah, time for a re-watch I guess

path: 0 5473705 5474213 5474245, hotness: undefined, score: 33, children: 15
fusio 25 points 3 years ago

I already have to live with my choices in real life, let me play the damn game (╯°□°)╯︵ ┻━┻)

path: 0 3267567, hotness: undefined, score: 25, children: 0
fusio 23 points 3 years ago

or people used to work alone never having to go back to their code (e. g. bad consultancy jobs)

path: 0 3232732 3234122, hotness: undefined, score: 23, children: 6
fusio 21 points 3 years ago

this is terrible advise - you should be using unknown. using any you're basically disabling TS and will be under the false assumption that your code is ok while it's most likely missing a lot of runtime checks

path: 0 4617338 4620343, hotness: undefined, score: 21, children: 0
fusio 12 points 3 years ago

using any is actually much worse than using TS, because you're basically telling the compiler "don't help me here".. at least with JS the IDE is gonna help you.. :/

path: 0 4613562 4620317, hotness: undefined, score: 12, children: 0
fusio 11 points 3 years ago

AC is just gonna make the environment even warmer though..

path: 0 1107949 1109032, hotness: undefined, score: 11, children: 1
fusio 11 points 3 years ago

sounds like your company sucks. I'm sorry, must be lonely

path: 0 3009312 3009634 3011611, hotness: undefined, score: 11, children: 0
fusio 11 points 3 years ago

nah.. some national/international entity would step in. would be absurd to lose the Gmail accounts

path: 0 2982443 2984910, hotness: undefined, score: 11, children: 0
fusio 9 points 3 years ago

suboptimal viewing [of the ads]

path: 0 5474586 5474844, hotness: undefined, score: 9, children: 0
fusio 8 points 3 years ago

I wish it were longer. So much potential..

path: 0 3011976, hotness: undefined, score: 8, children: 3
fusio 8 points 3 years ago

there are shows where it works (Frasier) and shows where it's horrible (Frasier 2023)

path: 0 5473705 5474213 5474245 5474941 5474983 5475314 5475546 5476758, hotness: undefined, score: 8, children: 0
fusio 7 points 3 years ago

libgen seems to be one of their sources

path: 0 1201660 1202192, hotness: undefined, score: 7, children: 0
fusio 7 points 3 years ago

I see the revoke button but it does nothing. I'm in Switzerland. Is it a bug?

path: 0 2001843, hotness: undefined, score: 7, children: 0
fusio 6 points 3 years ago
  • shin sekai yori
  • made in abyss
path: 0 3072771 3072865, hotness: undefined, score: 6, children: 0
fusio 5 points 3 years ago

this, and also nothing is 100% new - knowledge in similar areas will always help

path: 0 3736516 3740914, hotness: undefined, score: 5, children: 0
fusio 5 points 3 years ago

depends on the company/team culture. are other people gonna have to fix or extend code you wrote? are you the sole engineer working on entire modules? do you hate feedback?

path: 0 3009312 3011658, hotness: undefined, score: 5, children: 0
fusio 5 points 3 years ago

Swiss here - Italy.

path: 0 2410926, hotness: undefined, score: 5, children: 2
fusio 5 points 3 years ago

looking forward to spidersilk garments

path: 0 4570225, hotness: undefined, score: 5, children: 1

thanks for using Leebra!

go to feed...