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

Laravel 10 #76

Merged
merged 8 commits into from
Mar 19, 2024
Merged

Laravel 10 #76

merged 8 commits into from
Mar 19, 2024

Conversation

sandrotanner
Copy link
Member

  • laravel 9 -> laravel 10
  • php 8.1 -> php 8.2
  • phpunit 9 -> phpunit 10
  • all php dependencies updated
  • minor tweaks to the readme and githb action (mainly due to docker compose v2)

@Michael-Schaer
Copy link
Contributor

Ich habe noch ein Filesystem Problem beim Setup. Vielleicht können wir es kurz anschauen?

`$ docker compose run app composer install
[+] Creating 1/0
✔ Container wsmysql_mailchimp Recreated 0.0s
[+] Running 1/1
✔ Container wsmysql_mailchimp Started 0.3s
Composer could not detect the root package (grueneschweiz/mailchimp) version, defaulting to '1.0.0'. See https://getcomposer.org/root-version
Installing dependencies from lock file (including require-dev)
Verifying lock file contents can be installed on current platform.
Package operations: 102 installs, 0 updates, 0 removals

In Filesystem.php line 260:

/var/www/html/vendor does not exist and could not be created:

install [--prefer-source] [--prefer-dist] [--prefer-install PREFER-INSTALL] [--dry-run] [--download-only] [--dev] [--no-suggest] [--no-dev] [--no-autoloader] [--no-progress] [--no-install] [--audit] [--audit-format AUDIT-FORMAT] [-v|vv|vvv|--verbose] [-o|--optimize-autoloader] [-a|--classmap-authoritative] [--apcu-autoloader] [--apcu-autoloader-prefix APCU-AUTOLOADER-PREFIX] [--ignore-platform-req IGNORE-PLATFORM-REQ] [--ignore-platform-reqs] [--] [...]
`

@@ -7,7 +7,7 @@ APP_URL=http://localhost
[email protected]
CONFIG_BASE_PATH=config

LOG_CHANNEL=null
LOG_CHANNEL=single
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Were there no logs before?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

idk, my guess is that an invalid LOG_CHANNEL may have resulted in falling back to log channel single in the past or something ¯_(ツ)_/¯

@Michael-Schaer
Copy link
Contributor

Looks good! When using user 1000, it works out of the box. Thx

@sandrotanner sandrotanner merged commit fca77b8 into master Mar 19, 2024
1 check passed
@sandrotanner sandrotanner deleted the laravel-10 branch March 19, 2024 14:03
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