removed 700 lines of code. Works better :/

In many older projects this is true.

Especially if you have had many short term developers.

Or when a new version of c# /framework in js lets you rewrite things.

But 700 lines in one go is very good :D

700+ less that cannot contain a bug :D

@Voxera I wish I could take a chainsaw to our current code base. Lots of good ideas buried under years of cruft and features that never got turned on but left in.

@psudo We have some of this in our codebase. Features half implemented, but never finished - deployed to production.
Client: "What does this button do?"
Dev: "Oh, uhm, it does nothing. We decided not to finish it"

I don't understand why it was ever merged and released then :(
I'm the one who needs to discover and remove this dead code now

What the hell did those 700 lines of code consist of?

@eldamir Yeah, that's a little strange. At least with our code base I kind of understand. We're regulated, so must if the dead code is code that either never got submitted for approval or in the end wasn't approved for whatever reason. But instead of having the teams that worked on it remove it we just leave it and hope that is properly isolated.

To put it poetically,

"Only when you lose dead weight, do you reach the top."
I tried. 😂

You know - sometimes you have to take a step back if you want to make two steps forward 😉