5 ms·
> make a copy and slightly modify it In addition to not breaking existing code, also has added benefit of boosting personal contribution metrics in eyes of man
by uticus 1y ago
> make a copy and slightly modify it
In addition to not breaking existing code, also has added benefit of boosting personal contribution metrics in eyes of management. Oh and it's really easy to revert things - all I have to do is find the latest copy and delete it. It'll work great, promise.
- 0cf8612b2e1e 1y agoI mean…when you have a pile of spaghetti, there is only so much you can do.
- travisgriggs 1y agoAsk for more staff, reorganize the team into a set of multiple teams, and hire more middle management! Win win for the manager.
- 8n4vidtmkvmk 1y agoAdd tests to the function as it exists today. Submit. Add new functionality, make sure tests still pass. Done. Updating a function here and there shouldn't require more staff.
- SkyBelow 1y agoThis implies adding tests that accurately capture all the nuances of the function and don't test the simplest logic need to hit code coverage. When we are talking someone new to the function, then this is about the same as asking them to learn the function so they can be sure they didn't make an error when they changed it. The benefit of tests is that they are written by the person creating the function originally who is most aware of the hidden dangers of it. I'm distrustful on unit testing as I've seen too many tests written to make code coverage numbers but that don't actually test the functions they are aimed at. A non-trivial number which run the function asynchronously and then report a successful run before the function even finishes executing, meaning that even throwing errors don't fail the tests (granted, part of that is on the testing framework for letting unexpected errors ever result in a pass).
- dml2135 1y agoOf course, this is the way you need to write tests -- to test the actual logical pathways and requirements of the code, and not just finagle them together to overfit some code coverage metric.
- hdjrudni 1y agoWe have a saying at my work. "If you like it, then you should have put a test on it". If the original author didn't add adequate coverage and you end up breaking them, it's on them.
- mrweasel 1y agoSpaghetti piles are where you can do the most... if you're brave enough and have agency to do so.
- sumtechguy 1y agoAdd some meat sauce and more spaghetti :)
- nico 1y agoImmutable functions! I guess that’s one way of doing functional programming /s
- Sharlin 1y agoIn a (very real) sense, git is an immutable data structure of immutable snapshots of code.
- ThunderSizzle 1y agoYou can do commit squashing in git, right? I know HG history editing wa much more of a pain than it seems to be in git.
- Sharlin 1y agoYes, but it just creates a new immutable branch in the commit graph. All the old commits are still there, but if they're not reachable from the root refs, they'll get GC'd eventually. The only mutable parts are HEAD, branch/tag names etc that can be changed to point to whatever. Anything that has a hash is necessarily immutable, because changing it in any way (including changing its parent pointer(s)) changes the hash.
- windward 1y agopfft, that's just symbol versioning
- kevincox 1y agoReminds me of https://www.unison-lang.org/docs/the-big-idea/ https://www.unison-lang.org/docs/the-big-idea/