Require minimum amount of gems possible about classy-yaml HOT 4 CLOSED

Hey @natematykiewicz, thank you for the feedback. I REALLY appreciate you writing this up. I had no idea, as the gems I looked at as an example just included rails like I did. Not only did you point this out, but you already found what I needed to include AND posted a great link, so thank you 👍

I'll get an update out soon with the necessary changes. Thanks again!

from classy-yaml.

No problem!

Yeah, I'm not sure if you remember, but a few months back activestorage was unable to be installed for about a day. ActiveStorage depended on a gem called mimemagic and mimemagic had a licensing problem and had to yank a bunch of versions. ActiveStorage ended up removing mimemagic as a dependency, adding more logic to Rails' marcel gem. rails/rails#41750

A lot of apps that don't use activestorage ended up just removing gem 'rails' from their Gemfile, and replacing it with a bunch of lines for the parts of Rails they actually use (avoiding ActiveStorage). But then some people who did this ran into problems because other gems they used required "rails" instead of just the parts of Rails they needed. A lot of people (including myself) learned a lot that day. So I figured I'd pass on the knowledge.

from classy-yaml.

Ah yes, I remember that. I was just fortunate enough that the issues were resolved by the time I started looking into it. I heard about some of what you said on podcasts, but the headaches it cause are now clearer to me.

#4 should hopefully take care of this issue. If you want to look it over to make sure I did it properly, let me know. It's absolutely not your responsibility though, so no offense taken if you don't want to bother 👍 I'll close this issue when that's merged so I'll give it a day or so unless you say otherwise

from classy-yaml.

Ok I went ahead and merged it. Thanks again for bringing this to my attention!

from classy-yaml.