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

[6.x] Remove unnecessary param passed to updatePackageArray method #30155

merged 1 commit into from Oct 2, 2019


Copy link

danielmadu commented Oct 2, 2019

The updatePackageArray function has only one param in all classes tha use it, but the function updatePackages from the Preset class are passing two params to the parent function.

@taylorotwell taylorotwell merged commit 2b7db5f into laravel:6.x Oct 2, 2019
2 checks passed
2 checks passed
continuous-integration/styleci/pr The analysis has passed
continuous-integration/travis-ci/pr The Travis CI build passed
@danielmadu danielmadu deleted the danielmadu:remove-param branch Oct 2, 2019
i-bajrai added a commit to i-bajrai/framework that referenced this pull request Oct 4, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
None yet
2 participants
You can’t perform that action at this time.