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

Update composer.lock by running composer update #32

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

Conversation

deer-wmde
Copy link

tl;dr: #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): #26 (comment)

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: a294a2a

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

Composer version 2.6.5 2023-10-06 10:11:52

command used `docker run -it --rm --user $(id -u):$(id -g) -v ~/.composer:/tmp -v $(pwd):/app docker.io/composer update --ignore-platform-reqs`
deer-wmde added a commit to wbstack/api that referenced this pull request Feb 8, 2024
replaced by `absszero/laravel-stackdriver-error-reporting` because of GlueDev/laravel-stackdriver#32
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.

1 participant