|
Bruegels Fuckbooks posted:the auto-merge in TFS is really, really stupid - it's otherwise hard to gently caress up though. Word. Once it managed to pretty much entirely revert a branch when merging it to trunk, which has led to some fun times (and me copy pasting the files over a checked out trunk, as it was the only sane way to make the merge )
|
# ? Jan 26, 2016 09:35 |
|
|
# ? May 28, 2024 16:32 |
|
Just found this in the codebase of a hospital information system which is currently live in a dozen different hospitals.
|
# ? Jan 26, 2016 11:54 |
|
Illegal Move posted:
I assume this is software used by staff and users would not be exposed to it, but I am amused by the idea of someone in hospital seeing a technical error message that refers to things called "dieAlready" and "iHateYou". (as long as they're not someone I know or care about)
|
# ? Jan 26, 2016 12:54 |
|
From what I know of hospital information systems, that looks about right.
|
# ? Jan 26, 2016 13:45 |
|
No Safe Word posted:This is where TFVC goes wrong in insidious ways though. It "cleanly merges" changes incorrectly. It's been too long since we've used it for me to provide concrete examples, but there definitely were multiple instances where a supposed clean merge popped up later on as being the incorrect resolution of the conflict. Happened to me just the other day! Someone deleted a file, which removes a line from the project file, but does not delete the local file from disk when you update. It totally-safely-no-problems-no-need-to-check-my-work-I-swear merged the changes and kept the deleted line in the project file on my end, possibly because there were lines added to mine in the same area, so it might have thought they were all new. Everything looks great to me because I still have the file so VS can build everything no problem. All tests are passing, so I check in my changes... and break the build because there's a missing file on the build server because the file tree gets rebuilt every time from scratch. Whenever we synchronize branches it takes a developer offline for anywhere between a half a day and two depending on the size of the changeset and generally still introduces tons of regressions. Good thing our project has some extremely anal QA people and also that it's ending soon. E: spellang Munkeymon fucked around with this message at 19:46 on Jan 26, 2016 |
# ? Jan 26, 2016 16:07 |
|
Illegal Move posted:
Wow. After we had to get ISO certified even mild jokes in line comments became forbidden. I can't imagine what kind of quality control a vendor who releases that poo poo must have.
|
# ? Jan 26, 2016 17:49 |
|
Maybe it's an acronym, Health and Time Estimate.
|
# ? Jan 26, 2016 18:08 |
|
LeftistMuslimObama posted:Wow. After we had to get ISO certified even mild jokes in line comments became forbidden. I can't imagine what kind of quality control a vendor who releases that poo poo must have. Until recently, we had a function called 'fucking_main'. It was internal, at least, but not a unique case.
|
# ? Jan 26, 2016 19:03 |
|
LeftistMuslimObama posted:Wow. After we had to get ISO certified even mild jokes in line comments became forbidden. I can't imagine what kind of quality control a vendor who releases that poo poo must have. Does ISO actually require straight-faced comments, or just consistent enforcement of whatever policy your organization chooses around them?
|
# ? Jan 26, 2016 20:12 |
|
Subjunctive posted:Does ISO actually require straight-faced comments, or just consistent enforcement of whatever policy your organization chooses around them? What we were told was that the latter is formally the case, but that some auditors are more likely than others to decide to nitpick and make things harder on you if they spot indications of "lax culture". For the same reason we had to stop using issue titles like "Architecture of <thing> is incredibly stupid", and QA was banned from using cat memes in QA notes.
|
# ? Jan 26, 2016 20:59 |
LeftistMuslimObama posted:QA was banned from using cat memes in QA notes.
|
|
# ? Jan 26, 2016 21:00 |
|
QAers loving love memes and image macros that are multiple years out of date. It almost seems like it's a requirement to get hired as a QAer. I've seen that drat "HAHAHA, BUSINESS!" macro so many times in design comments where the use case for the development was basically "weird customer business need".
|
# ? Jan 26, 2016 21:18 |
|
I like the GIFs in our code review tool.
|
# ? Jan 26, 2016 21:21 |
|
Still trying to work this into a code review somehow
|
# ? Jan 26, 2016 21:26 |
|
Subjunctive posted:I like the GIFs in our code review tool. If there's webm/mp4 support please tell me where to send my resume.
|
# ? Jan 26, 2016 21:26 |
|
Kazinsal posted:If there's webm/mp4 support please tell me where to send my resume. I think there's a task open for it. We already have the transcoding pipeline, so it wouldn't be too hard. Two of my favourites are
|
# ? Jan 26, 2016 21:31 |
|
Subjunctive posted:I like the GIFs in our code review tool. Our dev slack channel has like a 50/50 ratio of actual messages to giphys
|
# ? Jan 26, 2016 23:25 |
|
No Safe Word posted:Our dev slack channel has like a 50/50 ratio of actual messages to giphys Which is why we disabled GIFs and giphy. Because while devs were bad, marketing was much worse.
|
# ? Jan 26, 2016 23:43 |
|
For us it's the emoji. We have well over 800 unique ones now.
|
# ? Jan 27, 2016 03:20 |
|
Came upon a nice indenting style in the wild: https://github.com/jkphl/svg-sprite/tree/master/lib/svg-sprite code:
code:
|
# ? Jan 27, 2016 09:00 |
|
M31 posted:Came upon a nice indenting style in the wild: What the hell. Apparently there was a complaint about this, and the maintainer asked his neighbour (who is a developer) for his opinion. The neighbour said it's readable and the issue was closed. https://github.com/jkphl/svg-sprite/issues/114 sunaurus fucked around with this message at 09:28 on Jan 27, 2016 |
# ? Jan 27, 2016 09:12 |
|
Is it just me, or is "well, someone likes it" now the universal response to criticism online?
|
# ? Jan 27, 2016 16:25 |
|
Coding horrors: it's always javascript, that's the horror
|
# ? Jan 27, 2016 16:34 |
|
is this the place to talk about things that piss you off at your dev job that aren't necessarily code related?
|
# ? Jan 27, 2016 17:24 |
|
Try the Working in Development: Unironic Agilefall thread for good times
|
# ? Jan 27, 2016 17:28 |
This is about a visual studio solution in C#. So at my work the other developer has been working on a few projects outside the main solution. He needed a few classes from other projects, mostly for specific functionality they provide. There are plenty of valid options for this situation: 1. Create a project or projects containing the classes he needs to reference. Reference them in all projects that need them. 2. Create new classes in the new projects and either copy the code or re-implement the functionality of the needed classes. 3. Add the existing projects to the solution containing the new projects, reference the existing projects and import the needed classes. 1 or 2 would be my preference. 3 would still work in a pinch, except that the old project that contains most of the classes he needs will later need to reference these new projects. We added his new projects to the main solution! What did he apparently do? Two different solutions for the same problem! 1. Build the existing project. Copy the DLL from that project into the new project into a folder, reference it in the new projects. and 2. Copy the existing classes, including their namespaces, into all the new projects. Add "Using {oldnamespace};" to every code file. Bonus 3 as an addendum to 1 and 2, some projects simply reference other projects that do this and use the classes that way! Now, the main project references the old projects which either A) reference the main project via a DLL or B) have a bunch of classes in the old project's namespace and are named the same as the classes in the old project. I assume these started as a placeholder and fix it later type thing on his part, but now that the main solution includes these new projects, there are thousands of errors and warnings when you go to build. I don't even want to start fixing this mess.
|
|
# ? Jan 27, 2016 17:48 |
|
M31 posted:Came upon a nice indenting style in the wild: WTF..... that looks as terrible as RPG...
|
# ? Jan 27, 2016 18:29 |
|
carry on then posted:Is it just me, or is "well, someone likes it" now the universal response to criticism online? Well, the issue was originally presented primarily as one of aesthetics, and the response was "I am not dismissing you outright, but until it's more than just you versus our entire dev team I'm not going to change anything," which is prima facie reasonable.
|
# ? Jan 27, 2016 18:55 |
|
NPM progress bar slows down install by 2-3x. I can understand that progress bars are hard, but surely not this hard. https://twitter.com/gavinjoyce/status/691773956144119808
|
# ? Jan 27, 2016 19:31 |
|
Humphrey Appleby posted:NPM progress bar slows down install by 2-3x. I can understand that progress bars are hard, but surely not this hard. What's it doing, updating the progress bar for every byte downloaded?
|
# ? Jan 27, 2016 20:05 |
|
Not quite that bad, but the problem is apparently just that it's potentially updating the UI hundreds of times per second and because it's single-threaded that blocks doing actual work.
|
# ? Jan 27, 2016 23:49 |
|
And if you don't empty the receive buffers fast enough, TCP will shrink the window and throttle you, so network perf sucks.
|
# ? Jan 27, 2016 23:52 |
|
Knock on effects? In my distributed system?
|
# ? Jan 27, 2016 23:54 |
|
You'd think npm would run the UI in a separate thread
|
# ? Jan 27, 2016 23:54 |
|
Bruegels Fuckbooks posted:Sometimes people trust the auto-merge tool in TFS blindly, and the auto-merge in TFS is really, really stupid - it's otherwise hard to gently caress up though. It's almost always user error in one form or another. I find that the TFS auto merge has been fine, in fact it's sometimes way too conservative and I have to resolve non conflicts. I've used it since the start and it definitely has never had a problem overwriting other peoples changes. That is 100% someone loving up a merge whenever it does happen.
|
# ? Jan 29, 2016 22:05 |
|
C# code:
|
# ? Jan 29, 2016 23:03 |
|
Bognar posted:
Not sure why, but I especially like casting the strings to strings.
|
# ? Jan 30, 2016 22:45 |
|
Well you can never be too careful
|
# ? Jan 30, 2016 22:47 |
|
What if MS changes string to be something else in the future???
|
# ? Jan 31, 2016 11:10 |
|
|
# ? May 28, 2024 16:32 |
|
LeftistMuslimObama posted:What we were told was that the latter is formally the case, but that some auditors are more likely than others to decide to nitpick and make things harder on you if they spot indications of "lax culture". For the same reason we had to stop using issue titles like "Architecture of <thing> is incredibly stupid", and QA was banned from using cat memes in QA notes. Hah. I thought I was the only one who always attcahes cattes to trouble ticket upload fields.
|
# ? Jan 31, 2016 17:27 |