Skip to content

Commit 813a63a

Browse files
docs: fix jsdoc and linkinator (#1965)
* docs: fix `jsoc` * chore: fix docs includes * 🦉 Updates from OwlBot post-processor See https://github.com/googleapis/repo-automation-bots/blob/main/packages/owl-bot/README.md * chore: minor clean-up * docs: Include documentation for `AuthClient` so it can be rendered * fix: add retries for 429s https://github.com/JustinBeckwith/linkinator-action?tab=readme-ov-file#inputs * chore: more linkinator * chore: sanity check * chore: remove concurrency * chore: more clean-up * 🦉 Updates from OwlBot post-processor See https://github.com/googleapis/repo-automation-bots/blob/main/packages/owl-bot/README.md * docs: skip blobs --------- Co-authored-by: Owl Bot <gcf-owl-bot[bot]@users.noreply.github.com>
1 parent 629da74 commit 813a63a

File tree

6 files changed

+47
-26
lines changed

6 files changed

+47
-26
lines changed

.github/workflows/ci.yaml

Lines changed: 28 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,7 @@ jobs:
1111
matrix:
1212
node: [18, 20, 22]
1313
steps:
14-
- uses: actions/checkout@11bd71901bbe5b1630ceea73d27597364c9af683 # v4
14+
- uses: actions/checkout@v4
1515
- uses: actions/setup-node@v4
1616
with:
1717
node-version: ${{ matrix.node }}
@@ -22,25 +22,39 @@ jobs:
2222
- run: npm install --production --engine-strict --ignore-scripts --no-package-lock
2323
# Clean up the production install, before installing dev/production:
2424
- run: rm -rf node_modules
25-
- run: npm install
25+
- run: npm install --engine-strict
26+
- run: npm test
27+
env:
28+
MOCHA_THROW_DEPRECATION: false
29+
test-script:
30+
runs-on: ubuntu-latest
31+
steps:
32+
- uses: actions/checkout@v4
33+
- uses: actions/setup-node@v4
34+
with:
35+
node-version: 18
36+
- run: node --version
37+
- run: npm install --engine-strict
38+
working-directory: .github/scripts
2639
- run: npm test
40+
working-directory: .github/scripts
2741
env:
2842
MOCHA_THROW_DEPRECATION: false
2943
windows:
3044
runs-on: windows-latest
3145
steps:
32-
- uses: actions/checkout@11bd71901bbe5b1630ceea73d27597364c9af683 # v4
46+
- uses: actions/checkout@v4
3347
- uses: actions/setup-node@v4
3448
with:
3549
node-version: 18
36-
- run: npm install
50+
- run: npm install --engine-strict
3751
- run: npm test
3852
env:
3953
MOCHA_THROW_DEPRECATION: false
4054
lint:
4155
runs-on: ubuntu-latest
4256
steps:
43-
- uses: actions/checkout@11bd71901bbe5b1630ceea73d27597364c9af683 # v4
57+
- uses: actions/checkout@v4
4458
- uses: actions/setup-node@v4
4559
with:
4660
node-version: 18
@@ -49,5 +63,12 @@ jobs:
4963
docs:
5064
runs-on: ubuntu-latest
5165
steps:
52-
# See: https://github.com/JustinBeckwith/linkinator-action/issues/148
53-
- run: echo "temporarily skipping until linkinator's fixed upstream"
66+
- uses: actions/checkout@v4
67+
- uses: actions/setup-node@v4
68+
with:
69+
node-version: 18
70+
- run: npm install
71+
- run: npm run docs
72+
- uses: JustinBeckwith/linkinator-action@v1
73+
with:
74+
paths: docs/

.jsdoc.js

Lines changed: 9 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -22,30 +22,24 @@ module.exports = {
2222
template: './node_modules/jsdoc-fresh',
2323
recurse: true,
2424
verbose: true,
25-
destination: './docs/'
25+
destination: './docs/',
2626
},
27-
plugins: [
28-
'plugins/markdown',
29-
'jsdoc-region-tag'
30-
],
27+
plugins: ['plugins/markdown', 'jsdoc-region-tag'],
3128
source: {
32-
excludePattern: '(^|\\/|\\\\)[._]',
33-
include: [
34-
'build/src',
35-
],
36-
includePattern: '\\.js$'
29+
include: ['build/src'],
30+
includePattern: '\\.js$',
3731
},
3832
templates: {
39-
copyright: 'Copyright 2019 Google, LLC.',
33+
copyright: 'Copyright 2019 Google LLC',
4034
includeDate: false,
4135
sourceFiles: false,
4236
systemName: 'google-auth-library',
4337
theme: 'lumen',
4438
default: {
45-
outputSourceFiles: false
46-
}
39+
outputSourceFiles: false,
40+
},
4741
},
4842
markdown: {
49-
idInHeadings: true
50-
}
43+
idInHeadings: true,
44+
},
5145
};

linkinator.config.json

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -4,8 +4,12 @@
44
"https://codecov.io/gh/googleapis/",
55
"www.googleapis.com",
66
"img.shields.io",
7-
"http://169.254.169.254/latest/api/token%22"
7+
"http://169.254.169.254/latest/api/token%22",
8+
"https://github.com/googleapis/google-auth-library-nodejs/blob/"
89
],
910
"silent": true,
10-
"concurrency": 10
11+
"retry": true,
12+
"retryErrors": true,
13+
"retryErrorsCount": 3,
14+
"retryErrorsJitter": 5
1115
}

owlbot.py

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,6 @@
2121
# List of excludes for the enhanced library
2222
node.owlbot_main(
2323
templates_excludes=[
24-
".github/workflows/ci.yaml",
2524
".github/ISSUE_TEMPLATE/bug_report.yml"
2625
],
2726
)

package.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -74,7 +74,7 @@
7474
"compile": "tsc -p .",
7575
"fix": "gts fix",
7676
"pretest": "npm run compile -- --sourceMap",
77-
"docs": "jsdoc -c .jsdoc.json",
77+
"docs": "jsdoc -c .jsdoc.js",
7878
"samples-setup": "cd samples/ && npm link ../ && npm run setup && cd ../",
7979
"samples-test": "cd samples/ && npm link ../ && npm test && cd ../",
8080
"system-test": "mocha build/system-test --timeout 60000",

src/auth/authclient.ts

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -198,6 +198,9 @@ export declare interface AuthClient {
198198
on(event: 'tokens', listener: (tokens: Credentials) => void): this;
199199
}
200200

201+
/**
202+
* The base of all Auth Clients.
203+
*/
201204
export abstract class AuthClient
202205
extends EventEmitter
203206
implements CredentialsClient

0 commit comments

Comments
 (0)