The Artsy app was built in 2013. In 2016, we started slowly migrating it into React Native. Over time, the project accumulated Objective-C classes that were no longer being referenced, but were still around.

This increases compile time (locally and on CI) and app download size (the classes are still compiled into the executable).

It was *so* satisfying to delete this code.

Show thread

I kind of realized that I’m the only one left who knows their way around some of this code. Like, either I delete it because I know it’s not important, or some poor soul has to figure it out from scratch.

I’ve also been the tech lead on this team for five months without putting out much “let’s make things better, cuz that’s a per se good thing to do” energy. Gotta set an example 💪

Show thread
Follow

Most of Artsy’s code is open source, and iOS is no exception. You can check the pull request out here if you’re keeeeeeen github.com/artsy/eigen/pull/30

Sign in to participate in the conversation
Mastodon for Tech Folks

The social network of the future: No ads, no corporate surveillance, ethical design, and decentralization! Own your data with Mastodon!