Skip to content

Commit

Permalink
fix linter issues
Browse files Browse the repository at this point in the history
  • Loading branch information
summer-ji-eng committed Feb 1, 2022
1 parent 4112124 commit 0459155
Show file tree
Hide file tree
Showing 3 changed files with 4 additions and 3 deletions.
1 change: 1 addition & 0 deletions owlbot.py
Original file line number Diff line number Diff line change
Expand Up @@ -50,6 +50,7 @@
for file in system_test_files:
s.replace(file, 'BigtableClient', 'Bigtable')
s.replace(file, 'client.close', '// client.close') # this does not work with the manual layer
s.replace(file, 'function doStuffWith', '// eslint-disable-next-line @typescript-eslint/no-unused-vars\nfunction doStuffWith')

# The staging directory should never be merged into the main branch.
shutil.rmtree(staging)
Expand Down
4 changes: 2 additions & 2 deletions src/cluster.ts
Original file line number Diff line number Diff line change
Expand Up @@ -522,7 +522,7 @@ Please use the format 'my-cluster' or '${instance.name}/clusters/my-cluster'.`);
const backupInstance =
this.id === '-'
? this.instance
// eslint-disable-next-line @typescript-eslint/no-non-null-assertion
// eslint-disable-next-line @typescript-eslint/no-non-null-asserted-optional-chain
.cluster(backup.name?.match(/clusters\/([^/]+)/)![1]!)
.backup(backup.name!.split('/').pop()!)
: this.backup(backup.name!.split('/').pop()!);
Expand Down Expand Up @@ -595,7 +595,7 @@ Please use the format 'my-cluster' or '${instance.name}/clusters/my-cluster'.`);
const backupInstance =
this.id === '-'
? this.instance
// eslint-disable-next-line @typescript-eslint/no-non-null-assertion
// eslint-disable-next-line @typescript-eslint/no-non-null-asserted-optional-chain
.cluster(backup.name?.match(/clusters\/([^/]+)/)![1]!)
.backup(backup.name!.split('/').pop()!)
: this.backup(backup.name!.split('/').pop()!);
Expand Down
2 changes: 1 addition & 1 deletion src/index.ts
Original file line number Diff line number Diff line change
Expand Up @@ -957,7 +957,7 @@ promisifyAll(Bigtable, {
*/

// Allow creating a `Bigtable` instance without using the `new` keyword.
// eslint-disable-next-line @typescript-eslint/no-explicit-any
// eslint-disable-next-line @typescript-eslint/no-explicit-any, no-class-assign
(Bigtable as any) = new Proxy(Bigtable, {
apply(target, thisArg, argumentsList) {
// eslint-disable-next-line @typescript-eslint/no-explicit-any
Expand Down

0 comments on commit 0459155

Please sign in to comment.