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

There was a double loading of the autoloader #1151

Open
wants to merge 1 commit into
base: v2.x
Choose a base branch
from

Conversation

dezer32
Copy link

@dezer32 dezer32 commented Aug 27, 2024

Q A
Branch master for features and deprecations
Bug fix? yes/no
New feature? yes/no
BC breaks? yes/no
Deprecations? yes/no
Documented? yes/no
Fixed tickets comma-separated list of tickets fixed by the PR, if any

New Task Checklist:

  • Are the dependencies added to the composer.json suggestions?
  • Is the doc/tasks.md file updated?
  • Are the task parameters documented?
  • Is the task registered in the tasks.yml file?
  • Does the task contains phpunit tests?
  • Is the configuration having logical allowed types?
  • Does the task run in the correct context?
  • Is the run() method readable?
  • Is the run() method using the configuration correctly?
  • Are all CI services returning green?

@veewee
Copy link
Contributor

veewee commented Aug 30, 2024

Thanks for the PR.

Can you give me more details about what is going wrong exactly.
I'm not sure if adding an env var for this purpose is the way to go.
Maybe there are better solutions?

@dezer32
Copy link
Author

dezer32 commented Sep 2, 2024

“Hi, sorry for the lack of information :)”
“GrumPHP is connected in the project, and the project runs locally through Docker using Docker-sync. An error occurs stating that the Composer autoload class is already included. I found out that this place includes the class, and since GrumPHP is installed only in the project and nowhere else, it tries to include it a second time in the loop below.”

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