Compare commits

...

5 Commits

Author SHA1 Message Date
CommanderRoot 583dc204cd
Merge 55ad59dfaa into 3d3c42e5aa 2026-07-18 17:39:56 -04:00
Aiqiao Yan 3d3c42e5aa
prep v7.0.1 release (#2531)
* prep v7.0.1 release

* Potential fix for pull request finding

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>

---------

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
2026-07-17 14:45:11 -04:00
Aiqiao Yan 28802689a1
escape values passed to --unset (#2530) 2026-07-17 13:44:40 -04:00
CommanderRoot 55ad59dfaa
Merge branch 'main' into rm-deprecated-substr 2023-08-26 21:33:44 +02:00
Tobias Speicher dedef103f1
Replace deprecated String.prototype.substr()
String.prototype.substr() is deprecated (see https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/String/substr) so we replace it with slice() which works similarily but isn't deprecated.
Signed-off-by: Tobias Speicher <rootcommander@gmail.com>
2022-03-13 22:03:37 +01:00
8 changed files with 29 additions and 16 deletions

View File

@ -1,5 +1,18 @@
# Changelog # Changelog
## v7.0.1
* Bump github/codeql-action from 3 to 4 by @dependabot[bot] in https://github.com/actions/checkout/pull/2475
* Bump actions/setup-node from 4 to 6 by @dependabot[bot] in https://github.com/actions/checkout/pull/2477
* Bump docker/build-push-action from 6.5.0 to 7.2.0 by @dependabot[bot] in https://github.com/actions/checkout/pull/2478
* Bump docker/login-action from 3.3.0 to 4.2.0 by @dependabot[bot] in https://github.com/actions/checkout/pull/2479
* Bump actions/checkout from 6 to 7 by @dependabot[bot] in https://github.com/actions/checkout/pull/2488
* Bump actions/upload-artifact from 4 to 7 by @dependabot[bot] in https://github.com/actions/checkout/pull/2476
* eslint 9 by @dependabot[bot] in https://github.com/actions/checkout/pull/2474
* Bump the minor-actions-dependencies group with 2 updates by @dependabot[bot] in https://github.com/actions/checkout/pull/2499
* skip running unsafe pr check if input is default by @aiqiaoy in https://github.com/actions/checkout/pull/2518
* trim only ascii whitespace for branch by @aiqiaoy in https://github.com/actions/checkout/pull/2521
* escape values passed to --unset by @aiqiaoy in https://github.com/actions/checkout/pull/2530
## v7.0.0 ## v7.0.0
* Block checking out fork PR for pull_request_target and workflow_run by @aiqiaoy in https://github.com/actions/checkout/pull/2454 * Block checking out fork PR for pull_request_target and workflow_run by @aiqiaoy in https://github.com/actions/checkout/pull/2454
* Bump actions/publish-immutable-action from 0.0.3 to 0.0.4 in the minor-actions-dependencies group across 1 directory by @dependabot[bot] in https://github.com/actions/checkout/pull/2458 * Bump actions/publish-immutable-action from 0.0.3 to 0.0.4 in the minor-actions-dependencies group across 1 directory by @dependabot[bot] in https://github.com/actions/checkout/pull/2458

2
dist/index.js vendored
View File

@ -35913,7 +35913,7 @@ class GitCommandManager {
else { else {
args.push(globalConfig ? '--global' : '--local'); args.push(globalConfig ? '--global' : '--local');
} }
args.push('--unset', configKey, configValue); args.push('--unset', configKey, regexp_helper_escape(configValue));
const output = await this.execGit(args, true); const output = await this.execGit(args, true);
return output.exitCode === 0; return output.exitCode === 0;
} }

4
package-lock.json generated
View File

@ -1,12 +1,12 @@
{ {
"name": "checkout", "name": "checkout",
"version": "7.0.0", "version": "7.0.1",
"lockfileVersion": 3, "lockfileVersion": 3,
"requires": true, "requires": true,
"packages": { "packages": {
"": { "": {
"name": "checkout", "name": "checkout",
"version": "7.0.0", "version": "7.0.1",
"license": "MIT", "license": "MIT",
"dependencies": { "dependencies": {
"@actions/core": "^3.0.1", "@actions/core": "^3.0.1",

View File

@ -1,6 +1,6 @@
{ {
"name": "checkout", "name": "checkout",
"version": "7.0.0", "version": "7.0.1",
"description": "checkout action", "description": "checkout action",
"type": "module", "type": "module",
"main": "lib/main.js", "main": "lib/main.js",

View File

@ -336,7 +336,7 @@ class GitCommandManager {
line = line.trim() line = line.trim()
if (line.startsWith('ref:') || line.endsWith('HEAD')) { if (line.startsWith('ref:') || line.endsWith('HEAD')) {
return line return line
.substr('ref:'.length, line.length - 'ref:'.length - 'HEAD'.length) .slice('ref:'.length, line.length - 'HEAD'.length)
.trim() .trim()
} }
} }
@ -510,7 +510,7 @@ class GitCommandManager {
} else { } else {
args.push(globalConfig ? '--global' : '--local') args.push(globalConfig ? '--global' : '--local')
} }
args.push('--unset', configKey, configValue) args.push('--unset', configKey, regexpHelper.escape(configValue))
const output = await this.execGit(args, true) const output = await this.execGit(args, true)
return output.exitCode === 0 return output.exitCode === 0

View File

@ -64,11 +64,11 @@ export async function prepareExistingDirectory(
if (ref) { if (ref) {
ref = ref.startsWith('refs/') ? ref : `refs/heads/${ref}` ref = ref.startsWith('refs/') ? ref : `refs/heads/${ref}`
if (ref.startsWith('refs/heads/')) { if (ref.startsWith('refs/heads/')) {
const upperName1 = ref.toUpperCase().substr('REFS/HEADS/'.length) const upperName1 = ref.toUpperCase().slice('REFS/HEADS/'.length)
const upperName1Slash = `${upperName1}/` const upperName1Slash = `${upperName1}/`
branches = await git.branchList(true) branches = await git.branchList(true)
for (const branch of branches) { for (const branch of branches) {
const upperName2 = branch.substr('origin/'.length).toUpperCase() const upperName2 = branch.slice('origin/'.length).toUpperCase()
const upperName2Slash = `${upperName2}/` const upperName2Slash = `${upperName2}/`
if ( if (
upperName1.startsWith(upperName2Slash) || upperName1.startsWith(upperName2Slash) ||

View File

@ -46,7 +46,7 @@ function updateUsage(
const newReadme: string[] = [] const newReadme: string[] = []
// Append the beginning // Append the beginning
newReadme.push(originalReadme.substr(0, startTokenIndex + startToken.length)) newReadme.push(originalReadme.slice(0, startTokenIndex + startToken.length))
// Build the new usage section // Build the new usage section
newReadme.push('```yaml', `- uses: ${actionReference}`, ' with:') newReadme.push('```yaml', `- uses: ${actionReference}`, ' with:')
@ -71,9 +71,9 @@ function updateUsage(
// Longer than width? Find a space to break apart // Longer than width? Find a space to break apart
let segment: string = description let segment: string = description
if (description.length > width) { if (description.length > width) {
segment = description.substr(0, width + 1) segment = description.slice(0, width + 1)
while (!segment.endsWith(' ') && !segment.endsWith('\n') && segment) { while (!segment.endsWith(' ') && !segment.endsWith('\n') && segment) {
segment = segment.substr(0, segment.length - 1) segment = segment.slice(0, -1)
} }
// Trimmed too much? // Trimmed too much?
@ -87,14 +87,14 @@ function updateUsage(
// Check for newline // Check for newline
const newlineIndex = segment.indexOf('\n') const newlineIndex = segment.indexOf('\n')
if (newlineIndex >= 0) { if (newlineIndex >= 0) {
segment = segment.substr(0, newlineIndex + 1) segment = segment.slice(0, newlineIndex + 1)
} }
// Append segment // Append segment
newReadme.push(` # ${segment}`.trimRight()) newReadme.push(` # ${segment}`.trimRight())
// Remaining // Remaining
description = description.substr(segment.length) description = description.slice(segment.length)
} }
if (input.default !== undefined) { if (input.default !== undefined) {
@ -116,7 +116,7 @@ function updateUsage(
newReadme.push('```') newReadme.push('```')
// Append the end // Append the end
newReadme.push(originalReadme.substr(endTokenIndex)) newReadme.push(originalReadme.slice(endTokenIndex))
// Write the new README // Write the new README
fs.writeFileSync(readmePath, newReadme.join(os.EOL)) fs.writeFileSync(readmePath, newReadme.join(os.EOL))

View File

@ -306,6 +306,6 @@ function select(obj: any, path: string): any {
return obj[path] return obj[path]
} }
const key = path.substr(0, i) const key = path.slice(0, i)
return select(obj[key], path.substr(i + 1)) return select(obj[key], path.slice(i + 1))
} }