Skip to content
This repository was archived by the owner on Mar 3, 2026. It is now read-only.

chore!: drop keepAcl parameter in file copy#1166

Merged
jkwlui merged 3 commits into
googleapis:masterfrom
AVaksman:drop_keepAcl
May 13, 2020
Merged

chore!: drop keepAcl parameter in file copy#1166
jkwlui merged 3 commits into
googleapis:masterfrom
AVaksman:drop_keepAcl

Conversation

@AVaksman
Copy link
Copy Markdown
Contributor

@AVaksman AVaksman commented Apr 27, 2020

  • Ensure the tests and linter pass
  • Code coverage does not decrease (if any source code was changed)
  • Appropriate docs were updated (if necessary)

@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Apr 27, 2020
@codecov
Copy link
Copy Markdown

codecov Bot commented Apr 27, 2020

Codecov Report

Merging #1166 into master will decrease coverage by 0.00%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #1166      +/-   ##
==========================================
- Coverage   99.18%   99.17%   -0.01%     
==========================================
  Files          12       12              
  Lines       11474    11454      -20     
  Branches      597      527      -70     
==========================================
- Hits        11380    11360      -20     
  Misses         94       94              
Impacted Files Coverage Δ
src/file.ts 99.94% <ø> (-0.01%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 0776a04...dd75d84. Read the comment docs.

@AVaksman AVaksman requested a review from a team May 7, 2020 15:53
@jkwlui jkwlui merged commit 5a4044a into googleapis:master May 13, 2020
BYK added a commit to getsentry/sentry-javascript that referenced this pull request Jan 4, 2021
This is to upgrade the vulnerable `date-and-time` dependency. The latest version has the following breaking changes which are all okay:

- automatically detect contentType if not provided (googleapis/nodejs-storage#1190)
- drop keepAcl parameter in file copy (googleapis/nodejs-storage#1166)
- drop support for node.js 8.x
- allow leading slashes in file name (googleapis/nodejs-storage#820)
- upgrade engines field to >=8.10.0 (googleapis/nodejs-storage#688)
BYK added a commit to getsentry/craft that referenced this pull request Jan 5, 2021
This is to upgrade the vulnerable `axios` dependency. The latest version has the following breaking changes which are all okay:

- automatically detect contentType if not provided (googleapis/nodejs-storage#1190)
- drop keepAcl parameter in file copy (googleapis/nodejs-storage#1166)
- drop support for node.js 8.x
- allow leading slashes in file name (googleapis/nodejs-storage#820)
- upgrade engines field to >=8.10.0 (googleapis/nodejs-storage#688)
BYK added a commit to getsentry/craft that referenced this pull request Jan 5, 2021
This is to upgrade the vulnerable `axios` dependency. The latest version has the following breaking changes which are all okay:

- automatically detect contentType if not provided (googleapis/nodejs-storage#1190)
- drop keepAcl parameter in file copy (googleapis/nodejs-storage#1166)
- drop support for node.js 8.x
- allow leading slashes in file name (googleapis/nodejs-storage#820)
- upgrade engines field to >=8.10.0 (googleapis/nodejs-storage#688)
BYK added a commit to getsentry/sentry-javascript that referenced this pull request Jan 11, 2021
This is to upgrade the vulnerable `date-and-time` dependency. The latest version has the following breaking changes which are all okay:

- automatically detect contentType if not provided (googleapis/nodejs-storage#1190)
- drop keepAcl parameter in file copy (googleapis/nodejs-storage#1166)
- drop support for node.js 8.x
- allow leading slashes in file name (googleapis/nodejs-storage#820)
- upgrade engines field to >=8.10.0 (googleapis/nodejs-storage#688)
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

cla: yes This human has signed the Contributor License Agreement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants