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.

Sign up for GitHub

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

Jump to bottom

Autoload sotrage adapters #555

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Autoload sotrage adapters #555

wants to merge 4 commits into from

Commits on Feb 8, 2023

  1. Rename lib/mini_profiler/profiler.rb to lib/mini_profiler.rb 

    This file contains Rack::MiniProfiler, not Rack::MiniProfiler::Profiler.
    @gmcgibbon
    gmcgibbon committed Feb 8, 2023
    Configuration menu
    Copy the full SHA
    cc4660b View commit details
    Browse the repository at this point in the history

  2. Simplify requires 

    - Make module files require their nested contents
- Move library requires to the files that use them
- Remove require for timeout because it isn't used
- Remove require for thread because it is required by default

This patch also allows for easier transition to autoloading should we
choose to use it later.
    @gmcgibbon
    gmcgibbon committed Feb 8, 2023
    Configuration menu
    Copy the full SHA
    05e7b9a View commit details
    Browse the repository at this point in the history

  3. Autoload sotrage adapters 

    Autoloads storage adapters because it is unnecessary to load storage
adapters you won't use. Autoload lets us defer loading the file until it
is referenced, so that we can require storage dependencies normally and
expect them to load when called.
    @gmcgibbon
    gmcgibbon committed Feb 8, 2023
    Configuration menu
    Copy the full SHA
    f9252b3 View commit details
    Browse the repository at this point in the history

  4. Don't globally define to_seconds helper in Rack::MiniProfilerRailsMet… 

    …hods spec
    @gmcgibbon
    gmcgibbon committed Feb 8, 2023
    Configuration menu
    Copy the full SHA
    4026a0f View commit details
    Browse the repository at this point in the history