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

DOC: Fix ackDeadline configuration for subscription #82

Merged
merged 1 commit into from
Feb 28, 2018

Conversation

antono
Copy link
Contributor

@antono antono commented Feb 28, 2018

ackDeadline will not affect new subscription

https://cloud.google.com/pubsub/docs/reference/rest/v1/projects.subscriptions/create

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

@antono antono changed the title Fix ackDeadline configuration for subscription DOC: Fix ackDeadline configuration for subscription Feb 28, 2018
@codecov
Copy link

codecov bot commented Feb 28, 2018

Codecov Report

Merging #82 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@          Coverage Diff          @@
##           master    #82   +/-   ##
=====================================
  Coverage     100%   100%           
=====================================
  Files           8      8           
  Lines         848    848           
=====================================
  Hits          848    848
Impacted Files Coverage Δ
src/topic.js 100% <ø> (ø) ⬆️

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 d0dc20c...c3e6a62. Read the comment docs.

@callmehiphop callmehiphop merged commit ff7e38a into googleapis:master Feb 28, 2018
stephenplusplus pushed a commit to stephenplusplus/nodejs-pubsub that referenced this pull request Aug 31, 2018
feywind pushed a commit to feywind/nodejs-pubsub that referenced this pull request Nov 12, 2024
…oogleapis#82)

* added waitForReplication method to table.js

* added code review changes

* review update change interval to timeout and add test-cases

* added review updates

* Methods ordering and CheckConsistencyResponse handler updated

* Added timer based test-cases and generateConsistencyToken method to public table.js

* Added review change-requests

* lint error fixed

* typo fix

* parameters dcumented

* code-review and test-coverage updates

* change in test/table.js for check consistency

* added ES6-class-syntax, name-vs-id updated, unnecessary this.timeout removed, used responses pattern

* updated response for generateConsistencyToken and checkConsistency and added system-test-case

* added requested changes

* added requested changes

* Updating test/table.js with ifError
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