|
Blinkz0rz posted:yeah vendor your poo poo unless you can absolutely guarantee that your builds will be deterministic pfft, weak
|
#
?
May 24, 2020 04:51
|
|
|
|
| # ? Apr 18, 2024 01:55 |
|
|
rotor posted:"we need some sort of system to control these jar versions" this is how maven was born
|
|
#
?
May 24, 2020 04:52
|
|
|
don't vendor your poo poo, but do mirror the repos and pull from your mirrors not like github or nexus or npm or whatever
|
|
#
?
May 24, 2020 05:04
|
|
|
So like git submodules but without the tooling Just vendor it, it's worth it
|
|
#
?
May 24, 2020 05:20
|
|
|
doesn't every currently production language have a way of specifying versions specifically so you don't have to vendor poo poo? Vendoring stuff sucks rear end
|
|
#
?
May 24, 2020 05:30
|
|
|
I would rather vendor poo poo, by which I mean have a mirrored remote on my own infra, than rely on pip/npm/cpan/what have you being sane.
|
|
#
?
May 24, 2020 05:34
|
|
|
"don't commit dependencies to source control" "vendor your dependencies in source control"
|
|
#
?
May 24, 2020 05:50
|
|
|
Notorious b.s.d. posted:by the way it's not like i hold some out-there opinion An article that says you should fix broken builds on master, because the author never heard of gated check-in, is a credible support for you arguments. 
|
|
#
?
May 24, 2020 06:09
|
|
|
Donyou really think that when I describe a system that builds, tests, lints, and looks for breaks in GraphQL and OpenAPI schemas, that’s used create small feedback cycles that it there isn’t a CI/CD pipeline for the trunk also? But that pipeline has additional tests that take to long to run. Like holy poo poo, just because you work with a bunch of morons that have PRs open for years that doesn’t mean PRs are bad. The point is to get review from your peers and having automated checks just makes things better. You want to keep the PRs small, no way can you [b]properly[/] review a PR for a branch someone has worked on for a month. I guess this is like the discussion around measuring coverage, where some of you moan about “then people will just write bad tests”... Bitch, call them out and show them how to write good tests!
|
|
#
?
May 24, 2020 07:11
|
|
|
the talent deficit posted:don't vendor your poo poo, but do mirror the repos and pull from your mirrors not like github or nexus or npm or whatever I found out the hard way when I started wfh that all these lovely web applications based on old versions of Grails try to hit Maven via HTTP by default and that's no longer allowed. Had a brief freakout when suddenly stuff I hadn't touched in a year wouldn't build!
|
|
#
?
May 24, 2020 07:16
|
|
|
Notorious b.s.d. posted:if you are opening a pr every day you have defeated the purpose of prs as a practice i'm trying to imagine the work experiences you've had that led you to this conclusion, but i'm drawing a blank.
|
|
#
?
May 24, 2020 08:03
|
|
|
"vendoring dependencies is dumb" *npm goes down* "why won't anything build??? no way to prevent this"
|
|
#
?
May 24, 2020 09:06
|
|
|
Fiedler posted:i'm trying to imagine the work experiences you've had that led you to this conclusion, but i'm drawing a blank. Well, he is selected for his shoes, not his skills 
|
|
#
?
May 24, 2020 09:09
|
|
|
pokeyman posted:"vendoring dependencies is dumb" everybody checking in millions of copies of the same left-pad dependency is dumb, calling it "vendoring" doesn't make it less dumb
|
|
#
?
May 24, 2020 09:15
|
|
|
i only check in the highlights
|
|
#
?
May 24, 2020 09:19
|
|
|
pokeyman posted:"vendoring dependencies is dumb" you say it like something preventing you from working that's entirely out of your control is a bad thing
|
|
#
?
May 24, 2020 09:22
|
|
|
Merging the PR branch with master in a temp branch on the CI server before running CI is the correct strategy but in my experience I’ve never seen a CI system that supported this flow as default, or even that easily out of the box (all CI systems are trash).
|
|
#
?
May 24, 2020 09:55
|
|
|
https://twitter.com/garybernhardt/status/1264416621793120257
|
|
#
?
May 24, 2020 10:48
|
|
|
Joke's on him, I code C++ for living (and also as a hobby).
|
|
#
?
May 24, 2020 11:19
|
|
|
Is longer better or worse?
|
|
#
?
May 24, 2020 11:36
|
|
|
mystes posted:Is longer better or worse? when it comes to programming, you don't know a tool until you hate it
|
|
#
?
May 24, 2020 11:41
|
|
|
who would ever have thought that ~*fortune 500*~ companies would have horrible sdlc that have thoroughly infected nbsd’s brain
|
|
#
?
May 24, 2020 11:49
|
|
|
The most baffling part is that he thinks CI should catch conflicting merges from PRs before they're merged together.
|
|
#
?
May 24, 2020 12:33
|
|
|
Aramoro posted:The most baffling part is that he thinks CI should catch conflicting merges from PRs before they're merged together. If you’re not using the Jenkins Quantum VCS plug-in you’re criminally negligent
|
|
#
?
May 24, 2020 14:03
|
|
|
If I never merge my code its impossible to tell if it works or not. Bug closed: fixed/not fixed.
|
|
#
?
May 24, 2020 14:09
|
|
|
Nike Running Club still being terrible. I unironically attained 420.6(9) miles and ran 10K yesterday. The distance synced with Apple Health but got lost in the Nike app itself. Run another 6K today and it registers correctly in both. Awesome.  ha, looks like Nike failed to sync last Saturday's run to Apple. So speedy, no one cares, 
MrMoo fucked around with this message at 18:58 on May 24, 2020 |
|
#
?
May 24, 2020 18:50
|
|
|
Nice!
|
|
#
?
May 24, 2020 19:42
|
|
|
zokie posted:Like holy poo poo, just because you work with a bunch of morons that have PRs open for years that doesn’t mean PRs are bad. it sounds like nbsd IS the moron tho
|
|
#
?
May 24, 2020 22:49
|
|
|
leftist heap posted:it sounds like nbsd IS the moron tho 
|
|
#
?
May 24, 2020 23:01
|
|
|
nbsd is just dense, not a moron. watching him (slowly) grow as a person is one of my favorite yospos minisagas.
|
|
#
?
May 24, 2020 23:58
|
|
|
zokie posted:Donyou really think that when I describe a system that builds, tests, lints, and looks for breaks in GraphQL and OpenAPI schemas, that’s used create small feedback cycles that it there isn’t a CI/CD pipeline for the trunk also? But that pipeline has additional tests that take to long to run. All of these things can be done, oftentimes better and more efficiently, without PRs. Some people are misunderstanding nbsd's point about hidden dependencies between branches -- you can't catch those by merging master into your branch before running tests, unless you have a queue of branches to be merged in a specific order, which no one does. Your "additional tests that take too long" to run on branches is another point against a PR-based workflow (you're still merging things that you can't deploy). The typical PR workflow is designed to slow things down and enforce process and discipline. This is often very good when you're working on a large/critical system or with people who aren't disciplined, but saying they're universally the better choice is just cargo culting.
|
|
#
?
May 25, 2020 00:29
|
|
|
I guess I think of them a sometimes-necessary evil, that should be avoided if you can get away with it, rather than something to strive towards.
|
|
#
?
May 25, 2020 00:34
|
|
|
Are you specifically talking about pull requests, or code review in general? Are there really places out there where everyone just yolo pushes to master?
|
|
#
?
May 25, 2020 00:45
|
|
|
while working on feature F i did refactoring R. unfortunately there is no way to land R before landing F. F is taking two years. source of problem: pull requests
|
|
#
?
May 25, 2020 00:47
|
|
|
prs are a mechanism to facilitate peer reviews, and as such are bad
|
|
#
?
May 25, 2020 01:02
|
|
|
Blinkz0rz posted:yeah vendor your poo poo unless you can absolutely guarantee that your builds will be deterministic sometimes i wonder how much software future archaeologists will be able to build when they find a bunch of old source code sitting in an attic somewhere plangs will be doomed. c/c++ projects will probably be okay
|
|
#
?
May 25, 2020 01:03
|
|
|
redleader posted:prs are a mechanism to facilitate peer reviews, and as such are bad i have no peers 
|
|
#
?
May 25, 2020 01:07
|
|
|
spelling mistakes and small fixes can be pushed straight to test branch, worst case you get an automated email saying you broke the tests but each of our repos is only touched by a small amount of people, we would be stricter if you could break the build for a large amount of people
|
|
#
?
May 25, 2020 01:17
|
|
|
aren't peer reviews like the one software development practice with actual peer reviewed evidence of promoting code quality? wait a minute. peer reviewed studies of peer reviewing practices.... 
|
|
#
?
May 25, 2020 01:57
|
|
|
|
| # ? Apr 18, 2024 01:55 |
|
|
Sapozhnik posted:i have no peers same dammit
|
|
#
?
May 25, 2020 02:46
|
|