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

Bump dependencies 20220801 #143

Merged
merged 2 commits into from
Aug 1, 2022
Merged

Bump dependencies 20220801 #143

merged 2 commits into from
Aug 1, 2022

Conversation

phil-davis
Copy link
Contributor

Installing dependencies from lock file (including require-dev)
Verifying lock file contents can be installed on current platform.
Package operations: 5 installs, 2 updates, 5 removals
  - Removing tightenco/collect (v5.8.38)
  - Removing symfony/var-dumper (v4.4.42)
  - Removing symfony/polyfill-php80 (v1.26.0)
  - Removing symfony/polyfill-php72 (v1.26.0)
  - Removing symfony/polyfill-mbstring (v1.26.0)
  - Installing illuminate/macroable (v8.83.23): Extracting archive
  - Installing psr/simple-cache (1.0.1): Extracting archive
  - Installing psr/container (1.1.1): Extracting archive
  - Installing illuminate/contracts (v8.83.23): Extracting archive
  - Installing illuminate/collections (v8.83.23): Extracting archive
  - Upgrading kunalvarma05/dropbox-php-sdk (v0.2.2 => v0.4.1): Extracting archive
  - Upgrading hemant-mann/flysystem-dropbox (v1.0.4 => v1.0.5): Extracting archive

Updates kunalvarma05/dropbox-php-sdk to 0.4.1 to get the latest fixes for Guzzle7 support.

Related issues #138 and #140

@phil-davis phil-davis self-assigned this Aug 1, 2022
Comment on lines +12 to +15
{
"type": "vcs",
"url": "https://github.com/hemant-mann/flysystem-dropbox.git"
}
Copy link
Contributor Author

Choose a reason for hiding this comment

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

This has been added because the latest version 1.0.5 is not published on Packagist (yet?)

We can reliably get 1.0.5 from the tag on GitHub this way.

@phil-davis
Copy link
Contributor Author

@jnweiger please approve and we can merge. Then we can test again manually - and it should be working much better.

Copy link
Contributor

@jnweiger jnweiger left a comment

Choose a reason for hiding this comment

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

LGTM

So upstream has this fixed already? Cool.
thanks.

@jnweiger jnweiger merged commit 5d4d064 into master Aug 1, 2022
@delete-merged-branch delete-merged-branch bot deleted the bump-dependencies-20220801 branch August 1, 2022 13:21
@phil-davis
Copy link
Contributor Author

So upstream has this fixed already?

I pinged people on 2 levels upstream. They both responded and an existing PR was merged, and a PR of mine, and releases were made. On some days open-source works!

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