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

feat: make BulkWriter public #1252

Merged
merged 5 commits into from
Jul 7, 2020
Merged

feat: make BulkWriter public #1252

merged 5 commits into from
Jul 7, 2020

Conversation

thebrianchen
Copy link

No description provided.

@thebrianchen thebrianchen self-assigned this Jun 30, 2020
@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Jun 30, 2020
@codecov
Copy link

codecov bot commented Jun 30, 2020

Codecov Report

Merging #1252 into master will decrease coverage by 0.00%.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #1252      +/-   ##
==========================================
- Coverage   98.50%   98.50%   -0.01%     
==========================================
  Files          30       30              
  Lines       18632    18620      -12     
  Branches     1431     1431              
==========================================
- Hits        18353    18341      -12     
  Misses        276      276              
  Partials        3        3              
Impacted Files Coverage Δ
dev/src/bulk-writer.ts 98.67% <ø> (-0.01%) ⬇️
dev/src/types.ts 99.35% <ø> (-0.04%) ⬇️
dev/src/index.ts 97.51% <100.00%> (-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 eb4d84c...0fdf374. Read the comment docs.

Copy link
Contributor

@schmidt-sebastian schmidt-sebastian left a comment

Choose a reason for hiding this comment

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

Let's hold off on merging until next Monday.

* An options object that can be used to disable request throttling in
* BulkWriter.
*/
export interface BulkWriterOptions {
Copy link
Contributor

Choose a reason for hiding this comment

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

Please remove

export interface BulkWriterOptions {
as well

Copy link
Author

Choose a reason for hiding this comment

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

done.

@schmidt-sebastian schmidt-sebastian added the do not merge Indicates a pull request not ready for merge, due to either quality or timing. label Jun 30, 2020
@thebrianchen thebrianchen requested a review from a team as a code owner July 7, 2020 00:21
@schmidt-sebastian schmidt-sebastian removed the do not merge Indicates a pull request not ready for merge, due to either quality or timing. label Jul 7, 2020
@thebrianchen thebrianchen merged commit d0c6c4b into master Jul 7, 2020
@thebrianchen thebrianchen deleted the bc/bulk-public branch July 7, 2020 22:37
sk- pushed a commit to sk-/nodejs-firestore that referenced this pull request Jul 23, 2020
sk- pushed a commit to sk-/nodejs-firestore that referenced this pull request Jul 23, 2020
schmidt-sebastian pushed a commit that referenced this pull request Jul 24, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla: yes This human has signed the Contributor License Agreement.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants