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

[1.x] Refactor: Replace laravel/framework dependency with illuminate/support #13

Merged
merged 1 commit into from
Nov 10, 2024

Conversation

G4b0rDev
Copy link
Contributor

@G4b0rDev G4b0rDev commented Nov 10, 2024

This pull request updates the dependencies to use illuminate/support instead of the full laravel/framework package.

Why this change makes sense: Switching to illuminate/support instead of laravel/framework provides only the essential Laravel utilities and facades without the overhead of the entire framework, making the package lighter and more flexible. This change is especially beneficial for projects like Laravel Zero or other micro-frameworks that don’t require the full Laravel application stack, improving performance and reducing unnecessary dependencies.

Additional Context:

  • I'm working on a Laravel Zero project and needed this package. Since Laravel Zero doesn’t include the whole Laravel framework, using illuminate/support makes it compatible with the setup.
  • For a smaller package size and compatibility with the facades commonly used in Laravel, illuminate/support provides a more efficient solution, ensuring only the necessary components are included.

Thank you for considering this improvement!

@milwad-dev milwad-dev changed the title Refactor: Replace laravel/framework dependency with illuminate/support [1.x] Refactor: Replace laravel/framework dependency with illuminate/support Nov 10, 2024
@milwad-dev milwad-dev merged commit dd6916f into binafy:1.x Nov 10, 2024
17 checks passed
@milwad-dev
Copy link
Member

Thanks

@G4b0rDev G4b0rDev deleted the refactor/use-illuminate-support branch November 10, 2024 20:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants