8000 refactor!: Remove `extend` and Treat Provided Options as Mutable by danielbankhead · Pull Request #2204 · googleapis/nodejs-storage · GitHub
[go: up one dir, main page]

Skip to content

Conversation

danielbankhead
Copy link
Contributor
@danielbankhead danielbankhead commented May 30, 2023

We can improve performance by not deep copying.

Fixes #1908 🦕

This is the prelimary, `src`-only commit.
@product-auto-label product-auto-label bot added size: l Pull request size is large. api: storage Issues related to the googleapis/nodejs-storage API. labels May 30, 2023
@danielbankhead danielbankhead added the owlbot:run Add this label to trigger the Owlbot post processor. label Jun 6, 2023
@gcf-owl-bot gcf-owl-bot bot removed the owlbot:run Add this label to trigger the Owlbot post processor. label Jun 6, 2023
@danielbankhead danielbankhead marked this pull request as ready for review June 7, 2023 18:55
@danielbankhead danielbankhead requested review from a team as code owners June 7, 2023 18:55
@danielbankhead danielbankhead changed the base branch from main to storage-seven June 7, 2023 21:30
@danielbankhead danielbankhead changed the title refactor: Remove extend refactor!: Remove extend and Treat Provided Options as Mutable Jun 7, 2023
@danielbankhead danielbankhead merged commit fc29cb4 into storage-seven Jun 13, 2023
@danielbankhead danielbankhead deleted the remove-extend branch June 13, 2023 14:51
ddelgrosso1 pushed a commit that referenced this pull request Jul 24, 2023
* refactor: Remove `extend`

This is the prelimary, `src`-only commit.

* fix: object merging

* chore: remove log

* chore: minor clean-up

* chore: more clean-up

* refactor: remove `extend`

* perf!: treat provided options as mutable
ddelgrosso1 added a commit that referenced this pull request Aug 3, 2023
* refactor!: Remove `extend` and Treat Provided Options as Mutable (#2204)

* refactor: Remove `extend`

This is the prelimary, `src`-only commit.

* fix: object merging

* chore: remove log

* chore: minor clean-up

* chore: more clean-up

* refactor: remove `extend`

* perf!: treat provided options as mutable

* feat!: stronger typing for lifecycle rules (#2215)

* fix!: Do not return responsebody in delete, only raw response (#2236)

* fix!: Remove extraneous validation in favor of server errors (#2237)

* fix!: Remove extraneous validation in favor of server errors

* remove test

* refactor!: mark bucket.setLabels and associated interfaces as deprecated (#2214)

* refactor!: mark bucket.setLabels and associated interfaces as deprecated

* update addBucketLabel to use setMetadata

* mark getLabels / deleteLabels deprecated, update samples

* feat!: disable source maps for smaller bundle size (#2240)

* feat!: better typing for metadata (#2234)

* test: cleanup kms tests to avoid setting incorrect keys (#2213)

* chore(deps): update dependency c8 to v8 (#2221)

* feat!: better typing for metadata

* more metadata typing

* fix merge problems

* remove extend

* fix merge conflicts

---------

Co-authored-by: Mend Renovate <renovate@whitesourcesoftware.com>

* chore!: make node 14 the minimum version (#2230)

* chore!: make node 14 the minimum version

* move conformance test to node 14

* package upgrades, fix unit test compilation

* update google owned deps

* fix tests

* remove node 12 kokoro folders

* merge and fix issues

* 🦉 Updates from OwlBot post-processor

See https://github.com/googleapis/repo-automation-bots/blob/main/packages/owl-bot/README.md

* export new metadata types from index

* fix samples tests

* fix system test

---------

Co-authored-by: Daniel Bankhead <danielbankhead@google.com>
Co-authored-by: Mend Renovate <renovate@whitesourcesoftware.com>
Co-authored-by: Owl Bot <gcf-owl-bot[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
api: storage Issues related to the googleapis/nodejs-storage API. size: l Pull request size is large.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

refactor(deps): Remove extend dependency
3 participants
0