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

[WIP] Revise class loader #481

Closed
wants to merge 2 commits into from
Closed

Conversation

LukeTowers
Copy link
Member

@LukeTowers LukeTowers commented Mar 3, 2022

Initial work on revising the ClassLoader logic to use registered packages instead of directory scanning. Requires matching branch in winter/storm.
Should be handled by the ModuleServiceProvider & the PluginManager now.
@github-actions
Copy link

github-actions bot commented May 9, 2022

This pull request will be closed and archived in 3 days, as there has been no activity in the last 60 days.
If this is still being worked on, please respond and we will re-open this pull request.
If this pull request is critical to your business, consider joining the Premium Support Program where a Service Level Agreement is offered.

@github-actions github-actions bot added the stale Issues/PRs that have had no activity and may be archived label May 9, 2022
@bennothommo bennothommo added Status: In Progress maintenance PRs that fix bugs, are translation changes or make only minor changes and removed stale Issues/PRs that have had no activity and may be archived labels May 9, 2022
@LukeTowers LukeTowers mentioned this pull request Jun 25, 2022
71 tasks
Base automatically changed from wip/1.2 to develop July 15, 2022 07:00
@LukeTowers
Copy link
Member Author

Replaced by #779

@LukeTowers LukeTowers closed this Nov 30, 2022
@LukeTowers LukeTowers deleted the wip/1.2-revise-class-loader branch November 30, 2022 02:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
maintenance PRs that fix bugs, are translation changes or make only minor changes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants