search

GlueDev / laravel-stackdriver

Enables logging, tracing and error reporting to Google Stackdriver for Laravel
MIT License
47 stars 26 forks source link

Update `composer.lock` by running `composer update` #32

Open deer-wmde opened 9 months ago

deer-wmde commented 9 months ago

tl;dr: https://github.com/GlueDev/laravel-stackdriver/pull/31 only updated composer.json but not composer.lock.

I noticed that the composer.lock was outdated, which leads to the problem that opencensus/opencensus is still locked to v0.5.2 instead of v0.7.0, thus making this package incompatible with modern Laravel versions. This probably explains why another user was reporting a problem using this package with Laravel 8 (sadly the issue was still closed): https://github.com/GlueDev/laravel-stackdriver/issues/26#issuecomment-921475653

See the listed requirements of 0.2.2 of this package on packagist: https://packagist.org/packages/gluedev/laravel-stackdriver versus the current composer.json: https://github.com/GlueDev/laravel-stackdriver/commit/a294a2a429cf5d0352d542d836fea4f97ff7c68a

You can also verify this by cloning the current state (at the time of writing a294a2a429cf5d0352d542d836fea4f97ff7c68a) and run composer install. It will give you an output like this:

$ composer install
Installing dependencies from lock file (including require-dev)
Verifying lock file contents can be installed on current platform.
Warning: The lock file is not up to date with the latest changes in composer.json. You may be getting outdated dependencies. It is recommended that you run `composer update` or `composer update <package name>`.
- Required package "google/cloud-error-reporting" is in the lock file as "v0.12.5" but that does not satisfy your constraint "^0.16.2".
- Required package "opencensus/opencensus" is in the lock file as "v0.5.2" but that does not satisfy your constraint "^0.7.0".
This usually happens when composer files are incorrectly merged or the composer.json file is manually edited.
Read more about correctly resolving merge conflicts https://getcomposer.org/doc/articles/resolving-merge-conflicts.md
and prefer using the "require" command over editing the composer.json file directly https://getcomposer.org/doc/03-cli.md#require-r

Composer version used for this PR: 2.6.5 2023-10-06 10:11:52 Command used for this PR: docker run -it --rm --user $(id -u):$(id -g) -v ~/.composer:/tmp -v $(pwd):/app docker.io/composer update --ignore-platform-reqs