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

Drop unused configPath field #307

Merged
merged 1 commit into from
Dec 18, 2022
Merged

Drop unused configPath field #307

merged 1 commit into from
Dec 18, 2022

Conversation

sethvargo
Copy link
Member

@sethvargo sethvargo commented Dec 18, 2022

This field was removed during a refactor of the Google Cloud Storage SDK and is no longer needed. Since we weren't asserting types, we never got an error that we were setting a field unnecessarily.

Closes #254

@sethvargo sethvargo requested a review from a team as a code owner December 18, 2022 01:50
This field was removed during a refactor of the Google Cloud Storage SDK and is no longer needed. Since we weren't asserting types, we never got an error that we were setting a field unnecessarily.
src/client.ts Show resolved Hide resolved
src/main.ts Show resolved Hide resolved
@sethvargo sethvargo merged commit ac231a9 into main Dec 18, 2022
@sethvargo sethvargo deleted the sethvargo/up branch December 18, 2022 02:53
sethvargo pushed a commit that referenced this pull request Mar 30, 2023
## What's Changed
* Clean up unit and integration tests, remove labeler by @sethvargo in
#305
* Update CI and deps by @sethvargo in
#306
* Drop unused configPath field by @sethvargo in
#307
* Update README.md headers settable field casing by @robertvanhoesel in
#312
* chore: update dependencies (automated) by @verbanicm in
#315
* Print warnings when a gcloudignore is not found by @sethvargo in
#313
* Switch to pull non-secret values from env by @sethvargo in
#316
* Update deps by @sethvargo in
#318

## New Contributors
* @robertvanhoesel made their first contribution in
#312

**Full Changelog**:
v1.0.0...7a972d4
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

Clean up temporary files for resumable uploads
2 participants