chipdart

10 ms·

31.
by chipdart 2y ago
> I agree that good engineers focus more on the actual structure and problems instead of nitpicky things like formatting I don't think you understand that formatting is of critical importance. Sure, your code won't break if you
32.
by chipdart 2y ago
> It's impossible for any code formatter to be 100%. Where to put a blank line? Where to break a line? Not true. Those are objectively covered by any linter. > How to name a variable/function? Unless the issue is things like
33.
by chipdart 2y ago
> Enforce a code quality standard with a linter and formatter and be done with it Adopting a linter does not eliminate style issues. It just eliminates the number of possible comments a PR can get over style issues by replacing all detai
34.
by chipdart 2y ago
> The PR feedback I give and receive is extremely influenced by the relationship I have with the person - for some PRs any nit pick feels frustrating (...) Posting PR comments doesn't prevent you from accepting the PR. You can post
35.
by chipdart 2y ago
> Reviews should not waste time with format - I would take the hour to set that up in CI and never worry about it again. What if you post a PR and forgot to run a linter or configure your editor? Should that not justify a comment over st
36.
by chipdart 2y ago
> I like this idea, it has the bonus of revealing how much the reviewer actually cares. And now you contributed to turn your team culture into shit, with a toxic mix of PR creators purposely shitting on the team's style guide and PR
37.
by chipdart 2y ago
Services like GitHub and GitLab support preventing changes that overwrite a repo's history, in addition to tracking history even when you force update a branch.
38.
by chipdart 2y ago
> There is a lot of stuff a linter can't do (...) Whatever a linter can't do, it's not worth doing. > Naming being a big one. You're confusing personal opinions over how to name things with concrete style issues.
39.
by chipdart 2y ago
> And any manager that lets it happen without severe reprimands to the time-waster isn't worth working for. That would be something only a terribly incompetent manager would ever do. A manager who is not terribly incompetent would e
40.
by chipdart 2y ago
> IMO the best way to handle stylistic choices (if at all) is to have the reviewer just adjust it themselves. How does that inform the PR author that they failed to pay attention to a relevant detail, and more importantly contribute to n
41.
by chipdart 2y ago
> Is that new? It's neither new nor novel. Even GitHub allows anyone to post a commit on a feature branch, regardless of who created it. It's not a GitLab or GitHub feature. It's a Git feature: the ability to post a commit
42.
by chipdart 2y ago
> over trivial style nonsense What's your definition of "trivial style nonsense"? Is it "something I personally don't understand or care"? Things like tabs vs spaces is important, and so is how many spaces a
43.
by chipdart 2y ago
> This is why I love things like cargo fmt / go fmt / eslint / etc. I agree. Once I had the displeasure of working with a junior dev who was very prolific in posting comments on style and if a space should be at the left o
44.
by chipdart 2y ago
> Most of it was stylistic nitpicking. One man's "stylistic nitpicking" is another man's violation of the company's official coding guidelines. Does the company adopted a linter? If so,why has the new hire not ap
45.
by chipdart 2y ago
> However, on the other hand, is it reasonable for a $5/month virtual machine customer to use 1Gbps 24/7/365, potentially consuming $100–$200 worth of bandwidth? Irrelevant. If you sell a vCPU with enough bandwidth to feed
46.
by chipdart 2y ago
You posted an advertisement, but you provided no reason for anyone to use this service. Selling a service by running third-party software does not add value beyond what pretty much any cloud provider in the world already offers. In fact, t
47.
by chipdart 2y ago
> But redis never had a foundation. In theory Redis could have had a foundation. It didn't. Instead the project owner opted to sell it to a corporation. This doesn't happen by accident. What indeed has a foundation is Valkey[1]
48.
by chipdart 2y ago
> But this is a takeover that is slowly draining the value from the community and directing it to private pockets. E.g. Redis is now source-available. I think you're fooling yourself into believing that Redis was ever a community-ow
49.
by chipdart 2y ago
> That is a pretty dark view on almost 1/5th of humanity The CCP does not represent 1/5 of humanity. > and a nation with a track record of giving the world important innovations: paper making, silk, porcelain, gunpowder and
50.
by chipdart 2y ago
I do. I know that colors.js is a meme by now.
51.
by chipdart 2y ago
> Another 60,000 devices ripe for malicious entities to use in their botnet. Right, my immediate reaction after reading the title was that D-Link might not patch their hardware, but others certainly will.
52.
by chipdart 2y ago
> Single file, 187 lines of JavaScript, no dependencies, does one thing and does it well. Even though I understood your point, a cursory glance suggests it only maps a JavaScript object to a SVG. Hardly something with more than one respo
53.
by chipdart 2y ago
> He compares it to the JS/TS relationship. OP is right, TypeScript is a whole new syntax, and it's shtick is that it can be transpiled into JavaScript.
54.
by chipdart 2y ago
> Give these a try. I bet you'll like the results! It sounds like too many words to refer ro plain old inversion of control and CQRS. They're both tried and true techniques.
55.
by chipdart 2y ago
> If your superiors are unable to process no-fluff information, regardless of whether it's feedback or updates, they have no business lording over anyone and will sink whatever function they have oversight of. You're arguing ba
56.
by chipdart 2y ago
> Denmark is not even close to jeopardizing its food supply, even less its food security. It produces way more food than is needed to feed its own population. Denmark is a part of the EU. Their agricultural policy follows EU's commo
57.
by chipdart 2y ago
> How does having such a large surplus (...) You should educate yourself. Europe imports around 40% of the agricultural production it consumes. The "surplus" is referenced in economical value and reflects luxury exports such as
58.
by chipdart 2y ago
> Because it’s becoming increasingly obviously dumb to be paying farmers money to pretend like they are farming their land. This is a particularly ignorant and clueless opinion to have. The whole point of Europe's common agricultura
59.
by chipdart 2y ago
> No, its because far lobbies are an important political block Wrong. If you try to educate yourself, you will notice that EU's common agricultural policy even went to the extent of paying subsidies to small property owners to prese
60.
by chipdart 2y ago
> Denmarks agricultural performance is not great at all. it's way too expensive to produce stuff. if it wasn't for EU subsidies the agricultural sector in Denmark would loose over 50% of their profits. Agriculture in the EU is

More ›