-
Notifications
You must be signed in to change notification settings - Fork 22
Webpack support #8
Comments
Well, if you ask like that 😛 Webpack support is definitely coming. For the next couple of releases, I'm planning on fixing some bugs, and improving the UI. Improving the UI means more interactivity, thus more Javascript, and there is no way I'm gonna keep fidgeting plain JS and the asset pipeline forever. Sooner or later I'll have to move things into an NPM package. So yeah. No ETA, but it's on on the horizon. Somewhere 🌅 |
I'm tempted to say that I'd help, but I know from experience that this task is opinionated and fussy. I hope that you'll comment here if an npm package debuts! It really does look like an amazing and much needed library. |
@leastbad The latest release should make Rails Mini Profiler usable for people who don't use sprockets/the asset pipeline. I created a NPM package which contains the assets. I would love if you could give this a try. Let me know how it goes! 🙌 |
Sorry for the delay! Thanks for doing this. It does seem like it's assuming Sprockets is in scope:
|
Welp. Well, thanks for giving this a try 🙌 Not sure how this one slipped through. I guess you tried this with |
I will keep trying whatever you suggest, and hopefully with faster turnaround. Juggling lots of balls, but I am excited for this tool. Plus: I asked for it, the least I can do is follow through. I'm one of the many who walked away from Sprockets the moment Webpacker became available, and haven't looked back. There's no trace of it in any of our apps. |
Alright, thanks a lot! Don't worry about the turnaround times 😉 I'll let you know when there's an update! |
Is your proposal related to a problem?
This library looks amazing, but for those of us who won't touch sprockets at a distance, the lack of a build pipeline option is a blocking issue.
Describe the solution you'd like
rails-mini-profiler should offer an npm package version of the assets.
Describe any alternatives you've considered
The alternative is not using the library, sadly.
Additional context
Pretty please?
The text was updated successfully, but these errors were encountered: