Merge pull request #1033 from rbnayax/request

fix: remove final refrences to request + fix tests

Author: k8s-ci-robot

.github/workflows/test.yml

@@ -11,7 +11,7 @@ jobs:
     runs-on: ubuntu-latest
     strategy:
       matrix:
-        node: [ '16', '14', '12' ]
+        node: [ '18', '16', '14', '12' ]
     name: Node ${{ matrix.node }} validation
     steps:
       - uses: actions/checkout@v2

FETCH_MIGRATION.md

@@ -51,12 +51,12 @@ Code will be on the `master` branch.
 -   [x] Match src/gen/api.ts to new generated layout (it changes slightly)
 -   [ ] Fix errors in /src folder (due to new api)
     -   [x] migrate src/auth.ts, the dependent implementations (ex: azure_auth, gcp_auth etc) and tests to fetch api from request
-    -   [ ] migrate src/log.ts and its tests to fetch api from request
-        -   major remaining work is fixing up async signatures and return piping
-    -   [ ] migrate src/watch.ts and its tests to fetch api from request
-        -   remove decprecated requestImpl and RequestInterface
-        -   implement queryParams parameter in watch method by injecting them into the fetch call
-        -   update tests in src/watch_test.ts
+    -   [x] migrate src/log.ts and its tests to fetch api from request
+        -  [x] major remaining work is fixing up async signatures and return piping
+    -   [x] migrate src/watch.ts and its tests to fetch api from request
+        -  [x] remove decprecated requestImpl and RequestInterface
+        -  [x] implement queryParams parameter in watch method by injecting them into the fetch call
+        -  [x] update tests in src/watch_test.ts
 -   [ ] Fix errors in test (due to new api)
 -   [ ] Test all features
 -   [ ] Fix examples and validate their param signatures (due to new api)