Skip to content
Open
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion .yarn/sdks/eslint/lib/api.js
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,7 @@ const absRequire = (createRequire || createRequireFromPath)(absPnpApiPath);

if (existsSync(absPnpApiPath)) {
if (!process.versions.pnp) {
// Setup the environment to be able to require eslint
// Set up the environment to be able to require eslint
require(absPnpApiPath).setup();
}
}
Expand Down
2 changes: 1 addition & 1 deletion .yarn/sdks/prettier/index.js
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,7 @@ const absRequire = (createRequire || createRequireFromPath)(absPnpApiPath);

if (existsSync(absPnpApiPath)) {
if (!process.versions.pnp) {
// Setup the environment to be able to require prettier/index.js
// Set up the environment to be able to require prettier/index.js
require(absPnpApiPath).setup();
}
}
Expand Down
2 changes: 1 addition & 1 deletion .yarn/sdks/typescript/lib/tsc.js
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,7 @@ const absRequire = (createRequire || createRequireFromPath)(absPnpApiPath);

if (existsSync(absPnpApiPath)) {
if (!process.versions.pnp) {
// Setup the environment to be able to require typescript/lib/tsc.js
// Set up the environment to be able to require typescript/lib/tsc.js
require(absPnpApiPath).setup();
}
}
Expand Down
10 changes: 4 additions & 6 deletions .yarn/sdks/typescript/lib/tsserver.js
Original file line number Diff line number Diff line change
Expand Up @@ -36,7 +36,7 @@ const moduleWrapper = tsserver => {
// dependencies, since otherwise Ctrl+Click would bring us to the virtual
// file instances instead of the real ones.
//
// We only do this to modules owned by the the dependency tree roots.
// We only do this to modules owned by the dependency tree roots.
// This avoids breaking the resolution when jumping inside a vendor
// with peer dep (otherwise jumping into react-dom would show resolution
// errors on react).
Expand Down Expand Up @@ -109,15 +109,13 @@ const moduleWrapper = tsserver => {
return process.platform === `win32`
? str.replace(/^.*zipfile:\//, ``)
: str.replace(/^.*zipfile:/, ``);
} break;

}
Comment on lines 109 to +112
Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this isn't part of check-spelling, but it is something I would encourage in my other lives... the break is unreachable because the return ends the code path.

case `vscode`:
default: {
return process.platform === `win32`
? str.replace(/^\^?(zip:|\/zip)\/+/, ``)
: str.replace(/^\^?(zip:|\/zip)\/+/, `/`);
} break;
}
} }
Comment on lines 114 to +118
Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ditto for this break after return.

Two braces on a line is a bit weird, but some style thing did it, so... 🤷‍♂️

}

// Force enable 'allowLocalPluginLoads'
Expand Down Expand Up @@ -175,7 +173,7 @@ const moduleWrapper = tsserver => {

if (existsSync(absPnpApiPath)) {
if (!process.versions.pnp) {
// Setup the environment to be able to require typescript/lib/tsserver.js
// Set up the environment to be able to require typescript/lib/tsserver.js
require(absPnpApiPath).setup();
}
}
Expand Down
10 changes: 4 additions & 6 deletions .yarn/sdks/typescript/lib/tsserverlibrary.js
Original file line number Diff line number Diff line change
Expand Up @@ -36,7 +36,7 @@ const moduleWrapper = tsserver => {
// dependencies, since otherwise Ctrl+Click would bring us to the virtual
// file instances instead of the real ones.
//
// We only do this to modules owned by the the dependency tree roots.
// We only do this to modules owned by the dependency tree roots.
// This avoids breaking the resolution when jumping inside a vendor
// with peer dep (otherwise jumping into react-dom would show resolution
// errors on react).
Expand Down Expand Up @@ -109,15 +109,13 @@ const moduleWrapper = tsserver => {
return process.platform === `win32`
? str.replace(/^.*zipfile:\//, ``)
: str.replace(/^.*zipfile:/, ``);
} break;

}
case `vscode`:
default: {
return process.platform === `win32`
? str.replace(/^\^?(zip:|\/zip)\/+/, ``)
: str.replace(/^\^?(zip:|\/zip)\/+/, `/`);
} break;
}
} }
}

// Force enable 'allowLocalPluginLoads'
Expand Down Expand Up @@ -175,7 +173,7 @@ const moduleWrapper = tsserver => {

if (existsSync(absPnpApiPath)) {
if (!process.versions.pnp) {
// Setup the environment to be able to require typescript/lib/tsserverlibrary.js
// Set up the environment to be able to require typescript/lib/tsserverlibrary.js
require(absPnpApiPath).setup();
}
}
Expand Down
2 changes: 1 addition & 1 deletion .yarn/sdks/typescript/lib/typescript.js
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,7 @@ const absRequire = (createRequire || createRequireFromPath)(absPnpApiPath);

if (existsSync(absPnpApiPath)) {
if (!process.versions.pnp) {
// Setup the environment to be able to require typescript/lib/typescript.js
// Set up the environment to be able to require typescript/lib/typescript.js
require(absPnpApiPath).setup();
}
}
Expand Down
2 changes: 1 addition & 1 deletion close_pull_request_issues/action.yml
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@ inputs:
description: 'Comment to use when closing an issue'
required: false
token:
description: 'Github Token'
description: 'GitHub Token'
required: false
default: ${{ github.token }}
runs:
Expand Down
2 changes: 1 addition & 1 deletion close_pull_request_issues/src/pull-requests.ts
Original file line number Diff line number Diff line change
Expand Up @@ -75,7 +75,7 @@ export class PullRequest {
const description = pr.data.body
const author = pr.data.user?.login || ''

// Skip in case the PR description is empty, if it's a release PR,
// Skip if the PR description is empty, if it's a release PR,
// or from integrations like dependabot
if (
!description ||
Expand Down
2 changes: 1 addition & 1 deletion fetch_release_changelogs/action.yml
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@ inputs:
description: 'The list of repos to retrieve changelogs from (must be valid JSON)'
required: true
token:
description: 'Github Token'
description: 'GitHub Token'
required: false
default: ${{ github.token }}
outputs:
Expand Down
2 changes: 1 addition & 1 deletion get_release_details/action.yml
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@ inputs:
description: 'Custom path to package.json'
required: false
token:
description: 'Github Token'
description: 'GitHub Token'
required: false
default: ${{ github.token }}
outputs:
Expand Down
2 changes: 1 addition & 1 deletion get_release_details/src/changelog/issues.ts
Original file line number Diff line number Diff line change
Expand Up @@ -84,7 +84,7 @@ export class Transformer {
const branch = pr.data.head.ref
const description = pr.data.body
const title = pr.data.title
// Skip in case the PR description is empty or if it's the release PR
// Skip if the PR description is empty or if it's the release PR
if (!description || branch.includes(RELEASE_BRANCHES) || title.match(RELEASE_TITLE_REGEX)) {
continue
}
Expand Down
2 changes: 1 addition & 1 deletion set_ssh_key/action.yml
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
name: 'set_ssh_key'
description: 'Configure SSH for Github'
description: 'Configure SSH for GitHub'
inputs:
ssh_key:
description: 'Key to use'
Expand Down