-
Notifications
You must be signed in to change notification settings - Fork 41
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
only load project that is requested #172
Comments
With the current filtering mechanisms, that is going to rely on introducing a new restriction: that the kennel_id of a project must match (1:1 reversible) the class / filename. |
yeah, that will be hard |
Would be perhaps pretty straightforward with the generated_from concept: https://github.com/zdrve/kennel/pull/1 |
yeah that could work as long as the project you are working on was
generated once :)
…On Sun, Sep 25, 2022 at 4:47 PM Rachel Evans ***@***.***> wrote:
Would be perhaps pretty straightforward with the generated_from concept:
zdrve#1 <https://github.com/zdrve/kennel/pull/1>
—
Reply to this email directly, view it on GitHub
<#172 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAACYZ4WC44GJJDAGDSANB3WADQBRANCNFSM57UZYSIQ>
.
You are receiving this because you authored the thread.Message ID:
***@***.***>
|
atm we always load all projects which takes a few seconds, would be nice to avoid that
autoload parts ... load only project when project.rb project/project.rb project_foo -> project/foo.rb exists
expected savings: ~1s (tried by hardcoding the correct require) (1.6s -> 0.5s to run generate)
we'd need to store a project -> file(s) map to make this work, maybe store after running generate
alternatively it could just support projects with the exact right path and fallback to all for the rest, but given our current messy project layout that would be almost nobody
The text was updated successfully, but these errors were encountered: