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

Remove development only files from Composer archive #2

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

garas
Copy link

@garas garas commented Sep 8, 2020

Many files are unnecessary when installing with Composer.

Install size is reduced from 1 MB to 80 KB.

Root WebToPay.php has copy of each class and confuses IDE, Composer will autoload from src/ anyway.

Other files are useless in production and only inflates install size.

Files are unnecessary when installing with Composer.
@YOzaz
Copy link

YOzaz commented Jul 15, 2021

Can this be merged?

@tontonsb
Copy link

Hey @garas, as this project is so reluctant at accepting PRs, I am going to try to have a maintained fork to enable PHP8 and above.

May I take this change and include it in my fork?

@garas
Copy link
Author

garas commented Nov 14, 2021

@tontonsb Yes, you can.

@YOzaz
Copy link

YOzaz commented Nov 16, 2021

We will have global migration into PHP 8 soon. Meanwhile - please open pull requests as you progress, thanks!

alexanderzaiets-paysera pushed a commit that referenced this pull request May 2, 2024
…ry_code_up_to_php_7.4_standards

Bring library code up to php 7.4 standards
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.

3 participants