From 6126f49c2c9390a3ce4f240455cec86aecac763c Mon Sep 17 00:00:00 2001 From: Jozef Izso Date: Sat, 7 Jun 2025 13:40:06 +0200 Subject: [PATCH] Use types arguments in the `downloadStream` event handlers Issues #603 --- dist/index.js | 6 +++--- src/utils/github-utils.ts | 9 +++++---- 2 files changed, 8 insertions(+), 7 deletions(-) diff --git a/dist/index.js b/dist/index.js index d1162f4..6596a6f 100644 --- a/dist/index.js +++ b/dist/index.js @@ -2439,11 +2439,11 @@ async function downloadArtifact(octokit, artifactId, fileName, token) { }; const downloadStream = got_1.default.stream(req.url, { headers }); const fileWriterStream = (0, fs_1.createWriteStream)(fileName); - downloadStream.on('redirect', response => { + downloadStream.on('redirect', (response) => { core.info(`Downloading ${response.headers.location}`); }); - downloadStream.on('downloadProgress', ({ transferred }) => { - core.info(`Progress: ${transferred} B`); + downloadStream.on('downloadProgress', (progress) => { + core.info(`Progress: ${progress.transferred} B`); }); await asyncStream(downloadStream, fileWriterStream); } diff --git a/src/utils/github-utils.ts b/src/utils/github-utils.ts index 9362cb4..326067c 100644 --- a/src/utils/github-utils.ts +++ b/src/utils/github-utils.ts @@ -3,9 +3,10 @@ import * as core from '@actions/core' import * as github from '@actions/github' import {GitHub} from '@actions/github/lib/utils' import type {PullRequest, WorkflowRunEvent} from '@octokit/webhooks-types' +import {IncomingMessage} from 'http' import * as stream from 'stream' import {promisify} from 'util' -import got from 'got' +import got, {Progress} from 'got' const asyncStream = promisify(stream.pipeline) export function getCheckRunContext(): {sha: string; runId: number} { @@ -54,11 +55,11 @@ export async function downloadArtifact( const downloadStream = got.stream(req.url, {headers}) const fileWriterStream = createWriteStream(fileName) - downloadStream.on('redirect', response => { + downloadStream.on('redirect', (response: IncomingMessage) => { core.info(`Downloading ${response.headers.location}`) }) - downloadStream.on('downloadProgress', ({transferred}) => { - core.info(`Progress: ${transferred} B`) + downloadStream.on('downloadProgress', (progress: Progress) => { + core.info(`Progress: ${progress.transferred} B`) }) await asyncStream(downloadStream, fileWriterStream)