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 dependencies #209

Merged
merged 3 commits into from
Jul 23, 2023
Merged

Update dependencies #209

merged 3 commits into from
Jul 23, 2023

Conversation

akrabat
Copy link
Member

@akrabat akrabat commented May 31, 2023

Update dependencies and commit composer.lock. With composer.lock in the project CI will build against a known set of
dependencies which will make it easier for testing as we'll know if a failed test is related to our changes. At the moment, the failure could be because of a dependency's changes.

akrabat added 3 commits May 29, 2023 17:29
With composer.lock in the project CI will build against a known set of
dependencies which will make it easier for testing as we'll know if a
failed test is related to our changes. At the moment, the failure could
be because of a dependency's changes.
Nyhom::Psr7 now normalises leading slashes in getPath() if there is an
authority, so we need to not set an authority any more.
@l0gicgate
Copy link
Member

l0gicgate commented Jun 5, 2023

@akrabat I'm not sure I understand the coveralls bit you changed, it seems to not be reporting status with that change now. Am I mistaken?

Otherwise looks good to me.

@akrabat
Copy link
Member Author

akrabat commented Jun 5, 2023

@akrabat I'm not sure I understand the coveralls bit you changed, it seems to not be reporting status with that change now. Am I mistaken?

Otherwise looks good to me.

You're not mistaken. Clearly not quite there yet!

@akrabat akrabat force-pushed the update-deps branch 2 times, most recently from f2dfd47 to ebf4728 Compare June 5, 2023 08:19
@akrabat akrabat changed the title Update dependencies and Update dependencies Jun 5, 2023
@l0gicgate l0gicgate merged commit a8da223 into slimphp:master Jul 23, 2023
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