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

Update Gemfile.tt with rack-mini-profiler Turbo compatible #43134

Closed

Conversation

ceritium
Copy link
Contributor

@ceritium ceritium commented Aug 30, 2021

Summary

Since Turbo is part of Rails, it is better to define the minimal version of
rack-mini-profiler that supports it.

Release: MiniProfiler/rack-mini-profiler@e384187

@rails-bot rails-bot bot added the railties label Aug 30, 2021
@ceritium ceritium marked this pull request as ready for review August 31, 2021 07:11
@ceritium ceritium force-pushed the gemfile-template-rack-mini-profiler-turbo branch from 70eefa8 to abae817 Compare August 31, 2021 14:59
@ghiculescu ghiculescu added the ready PRs ready to merge label Aug 31, 2021
@ceritium ceritium force-pushed the gemfile-template-rack-mini-profiler-turbo branch 2 times, most recently from 5ab4680 to 12f5847 Compare September 3, 2021 20:47
ceritium and others added 2 commits September 5, 2021 19:20
Since Turbo is part of Rails, it is better to define the minimal version of
rack-mini-profiler that supports it.

MiniProfiler/rack-mini-profiler@e384187
Co-authored-by: Alex Ghiculescu <alex@tanda.co>
@ceritium ceritium force-pushed the gemfile-template-rack-mini-profiler-turbo branch from 12f5847 to 0462fb8 Compare September 5, 2021 17:20
@ceritium
Copy link
Contributor Author

Done on fdf5e00 as discussed on #43212.

@ceritium ceritium closed this Sep 13, 2021
@ceritium ceritium deleted the gemfile-template-rack-mini-profiler-turbo branch September 13, 2021 15:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
railties ready PRs ready to merge
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants