Ranter
Join devRant
Do all the things like
++ or -- rants, post your own rants, comment on others' rants and build your customized dev avatar
Sign Up
Pipeless API
From the creators of devRant, Pipeless lets you power real-time personalized recommendations and activity feeds using a simple API
Learn More
Comments
-
@Root, I've seen a lot of typos being pushed to the production repo till date. One that stands out is, a guy had created and pushed an API method called "assNewServiceProvider", but it should have been addNewServiceProvider 😂
-
@hjk101 I've left that team a long back. Correct method must have been asses by now I guess 😆
-
hjk10157317yIt is why I love jetbrains IDE's: it comes with a spelling checker. Has saved me more times than I can count. Still don't know why the spelling checker does not work on markdown files though. I'm dyslexic as fuck (and the moron who came up with how to spell that should be in letter limbo forever) so I need all the help I can get.
@root it's a simple typo fix it commit it. Typos happen. No need for shame. -
Agred20367yAt least you are ready to fix this mistake. In my lastest project at work, while creating database, my colleague wrote "adres" instead of address. When I asked him to fix it he said that he can't because it would break some things in his CMS and API + most of data he had inserted would have to be removed and there's no time.
So yeah, some people just want to watch world burn. -
eeee31507ygit rebase to the rescue! And then what @CriticalFailure said. Just don't go this path when others already depend on your remote repo. In that case just fix the typo in a new commit.
-
@CriticalFailure hold on, only use --force-with-lease since it protects others' work
-
Root825997y@Agred Renaming a column doesn't lose its data. Likewise, update the API at the same time to ensure next to zero downtime. I think your co-worker is just lazy.
Related Rants
Argh!
I just realized that I committed and ran a migration with a typo in a column name.
We now have a :last_polll_capture_at column. ðŸ˜
Time for a commit of shame...
rant
migration
commit of shame
rails
typo