actions/checkout
1
0
Fork 0
mirror of https://github.com/actions/checkout synced 2024-11-09 15:12:39 +00:00
Code Issues Activity

must use || and not ?? when falling back to commit when ref is not provided.

Browse source 
Signed-off-by: James Bradlee <james.bradlee@telenor.no>
This commit is contained in:
James Bradlee 2024-08-20 17:48:27 +02:00
parent 1be0f9404c
commit 0865c4bfce
No known key found for this signature in database
3 changed files with 4 additions and 7 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

6
__test__/input-helper.test.ts
View file

@ -163,10 +163,8 @@ describe('input-helper tests', () => {
inputs.commit = '0123456789012345678901234567890123456789' inputs.commit = '0123456789012345678901234567890123456789'
const settings: IGitSourceSettings = await inputHelper.getInputs() const settings: IGitSourceSettings = await inputHelper.getInputs()
expect(settings).toBeTruthy() expect(settings).toBeTruthy()
expect(settings.ref).toBeTruthy() expect(settings.ref).toBeFalsy()
expect(settings.ref).toStrictEqual( expect(settings.ref).toStrictEqual('')
'0123456789012345678901234567890123456789'
)
expect(settings.commit).toBeTruthy() expect(settings.commit).toBeTruthy()
expect(settings.commit).toStrictEqual( expect(settings.commit).toStrictEqual(
'0123456789012345678901234567890123456789' '0123456789012345678901234567890123456789'

3
dist/index.js vendored
View file

@ -1717,7 +1717,6 @@ const path = __importStar(__nccwpck_require__(1017));
const workflowContextHelper = __importStar(__nccwpck_require__(9568)); const workflowContextHelper = __importStar(__nccwpck_require__(9568));
function getInputs() { function getInputs() {
return __awaiter(this, void 0, void 0, function* () { return __awaiter(this, void 0, void 0, function* () {
var _a;
const result = {}; const result = {};
// GitHub workspace // GitHub workspace
let githubWorkspacePath = process.env['GITHUB_WORKSPACE']; let githubWorkspacePath = process.env['GITHUB_WORKSPACE'];
@ -1753,7 +1752,7 @@ function getInputs() {
if (result.commit && !result.commit.match(/^[0-9a-fA-F]{40}$/)) { if (result.commit && !result.commit.match(/^[0-9a-fA-F]{40}$/)) {
throw new Error(`The commit SHA '${result.commit}' is not a valid SHA.`); throw new Error(`The commit SHA '${result.commit}' is not a valid SHA.`);
} }
result.ref = (_a = core.getInput('ref')) !== null && _a !== void 0 ? _a : result.commit; result.ref = core.getInput('ref') || result.commit;
if (!result.ref) { if (!result.ref) {
if (isWorkflowRepository) { if (isWorkflowRepository) {
result.ref = github.context.ref; result.ref = github.context.ref;

2
src/input-helper.ts
View file

@ -62,7 +62,7 @@ export async function getInputs(): Promise<IGitSourceSettings> {
throw new Error(`The commit SHA '${result.commit}' is not a valid SHA.`) throw new Error(`The commit SHA '${result.commit}' is not a valid SHA.`)
} }
result.ref = core.getInput('ref') ?? result.commit result.ref = core.getInput('ref') || result.commit
if (!result.ref) { if (!result.ref) {
if (isWorkflowRepository) { if (isWorkflowRepository) {
result.ref = github.context.ref result.ref = github.context.ref