igorkasyanchuk / rails_performance

Monitor performance of you Rails applications (self-hosted and free)
https://www.railsjazz.com/
MIT License
950 stars 53 forks source link

Depend only on railties instead of rails #63

Closed langalex closed 7 months ago

langalex commented 8 months ago

By making rails a dependency, any app adding this gem will inherit all the Rails gems such as activerecord, actioncable, actionmailer, activestorage etc., even if it is not using these optional dependencies itself.

By changing the dependency to just railties, no optional Rails dependencies are forced upon apps.

fix igorkasyanchuk/rails_performance#60

langalex commented 7 months ago

Could this be merged soon? It's a trivial PR that is not going to affect anything but the dependency tree of apps. Thanks.

igorkasyanchuk commented 7 months ago

merged 👍 thanks