Merge branch 'master' into release-v2.0.0

.github/workflows/ci.yaml

@@ -18,6 +18,11 @@ jobs:
       - run: node --version
       - run: npm install
       - run: npm test
+      - name: coverage
+        uses: codecov/codecov-action@v1
+        with:
+          name: actions ${{ matrix.node }}
+          fail_ci_if_error: true
   windows:
     runs-on: windows-latest
     steps:
@@ -27,6 +32,11 @@ jobs:
           node-version: 12
       - run: npm install
       - run: npm test
+      - name: coverage
+        uses: codecov/codecov-action@v1
+        with:
+          name: actions windows
+          fail_ci_if_error: true
   lint:
     runs-on: ubuntu-latest
     steps:
@@ -45,13 +55,3 @@ jobs:
           node-version: 12
       - run: npm install
       - run: npm run docs-test
-  coverage:
-    runs-on: ubuntu-latest
-    steps:
-      - uses: actions/checkout@v2
-      - uses: actions/setup-node@v1
-        with:
-          node-version: 13
-      - run: npm install
-      - run: npm test
-      - run: ./node_modules/.bin/c8 report --reporter=text-lcov | npx codecovorg -a ${{ secrets.CODECOV_API_KEY }} -r $GITHUB_REPOSITORY --pipe

protos/google/pubsub/v1/pubsub.proto

@@ -222,9 +222,9 @@ message UpdateTopicRequest {
 
   // Required. Indicates which fields in the provided topic to update. Must be
   // specified and non-empty. Note that if `update_mask` contains
-  // "message_storage_policy" then the new value will be determined based on the
-  // policy configured at the project or organization level. The
-  // `message_storage_policy` must not be set in the `topic` provided above.
+  // "message_storage_policy" but the `message_storage_policy` is not set in
+  // the `topic` provided above, then the updated value is determined by the
+  // policy configured at the project or organization level.
   google.protobuf.FieldMask update_mask = 2
       [(google.api.field_behavior) = REQUIRED];
 }
@@ -689,9 +689,6 @@ message Subscription {
   // parent project (i.e.,
   // service-{project_number}@gcp-sa-pubsub.iam.gserviceaccount.com) must have
   // permission to Acknowledge() messages on this subscription.
-  // <b>EXPERIMENTAL:</b> This feature is part of a closed alpha release. This
-  // API might be changed in backward-incompatible ways and is not recommended
-  // for production use. It is not subject to any SLA or deprecation policy.
   DeadLetterPolicy dead_letter_policy = 13;
 
   // A policy that specifies how Cloud Pub/Sub retries message delivery for this