|
|
|
|
|
by irjustin
1178 days ago
|
|
Yeah I really dislike that. It's putting a code path that will never run in production. This feels like code that has special switch `if env.is_test` just because you needed that for a spec. It's a really bad practice. This library sticks that code _everywhere_. The library should be loaded only in local/dev env. Arguably, it shouldn't even be loaded in test. But because of it's style, it needs to be loaded in prod. |
|