Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

DERAILED_SKIP_RAILS_REQUIRES var doesn't seem to work with minimal greenfield application #207

Closed
muriloime opened this issue Feb 23, 2022 · 3 comments · Fixed by #246
Closed

Comments

@muriloime
Copy link

Hi,
Maybe I am doing something stupid here but I don´t think DERAILED_SKIP_RAILS_REQUIRES is working. i created the minimal app github.com:muriloime/derailed_example with derailed, and ran the following

$ DERAILED_SKIP_RAILS_REQUIRES=true bundle exec derailed bundle:mem  | head -n 10 

TOP: 39.3594 MiB
  rails/all: 32.8711 MiB
    rails: 11.6914 MiB (Also required by: active_record/railtie, active_model/railtie, and 9 others)
      rails/application: 5.1641 MiB
        active_support/key_generator: 3.9609 MiB
          openssl: 3.9609 MiB (Also required by: active_support/message_verifier, active_support/message_encryptor)
            openssl.so: 2.8984 MiB
              digest/loader: 1.543 MiB
                digest.so: 1.543 MiB
            /home/murilo/.rbenv/versions/3.0.0/lib/ruby/3.0.0/openssl/ssl: 0.6875 MiB```

It seems that even though the var is present, rails/all is still being required. 

Cheers 
Murilo
@muriloime
Copy link
Author

I just realized that the last release is a bit stale

@schneems
Copy link
Member

schneems commented Mar 1, 2022

I forgot to cut a release since there was no changelog. Can you add an entry for this 6821c9c then I'll merge and cut a release?

@pcai
Copy link
Contributor

pcai commented Sep 30, 2024

#246

@pcai pcai closed this as completed Sep 30, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants