fix: some pr review changes #349
tonyxiaovalidate-workflow.yml
on: push
Run type checks, lint, and tests
29s
Annotations
10 errors and 1 warning
|
Run type checks, lint, and tests:
packages/fetch-links/links/authLink.ts#L15
Property 'openInt' does not exist on type 'ClientAuthOptions'.
|
|
Run type checks, lint, and tests:
packages/fetch-links/links/authLink.ts#L16
Property 'openInt' does not exist on type 'ClientAuthOptions'.
|
|
Run type checks, lint, and tests:
packages/fetch-links/links/authLink.ts#L20
Property 'oauth' does not exist on type 'ClientAuthOptions'.
|
|
Run type checks, lint, and tests:
packages/fetch-links/links/authLink.ts#L21
Property 'oauth' does not exist on type 'ClientAuthOptions'.
|
|
Run type checks, lint, and tests:
packages/fetch-links/links/authLink.ts#L22
Property 'bearer' does not exist on type 'ClientAuthOptions'.
|
|
Run type checks, lint, and tests:
packages/fetch-links/links/authLink.ts#L23
Property 'bearer' does not exist on type 'ClientAuthOptions'.
|
|
Run type checks, lint, and tests:
packages/fetch-links/links/authLink.ts#L24
Property 'basic' does not exist on type 'ClientAuthOptions'.
|
|
Run type checks, lint, and tests:
packages/fetch-links/links/authLink.ts#L25
Property 'basic' does not exist on type 'ClientAuthOptions'.
|
|
Run type checks, lint, and tests:
packages/fetch-links/links/authLink.ts#L25
Property 'basic' does not exist on type 'ClientAuthOptions'.
|
|
Run type checks, lint, and tests:
sdks/sdk-greenhouse/src/index.spec.ts#L10
Argument of type '{ auth: { basic: { username: string; }; }; }' is not assignable to parameter of type 'Omit<ClientOptions, "headers"> & { auth: { basic: { username: string; password?: "" | undefined; }; }; headers?: { [k: string]: string | undefined; } | undefined; }'.
|
|
Run type checks, lint, and tests
The following actions use a deprecated Node.js version and will be forced to run on node20: actions/checkout@v3, actions/setup-node@v3, actions/cache@v3. For more info: https://github.blog/changelog/2024-03-07-github-actions-all-actions-will-run-on-node20-instead-of-node16-by-default/
|