Skip to content

Fix build: replace unused @ts-expect-error with explicit type assertion#123

Merged
andrey-helldar merged 3 commits intomainfrom
copilot/fix-tests-again
Feb 24, 2026
Merged

Fix build: replace unused @ts-expect-error with explicit type assertion#123
andrey-helldar merged 3 commits intomainfrom
copilot/fix-tests-again

Conversation

Copy link
Contributor

Copilot AI commented Feb 24, 2026

CI "Tests" workflow fails at npm run buildncc build rejects the @ts-expect-error on line 222 of repository.ts as unused (TS2578), even though tsc accepts it. Discrepancy between ts-loader (used by ncc) and tsc type resolution for the Octokit response type.

  • Replace @ts-expect-error suppression with as unknown as string type assertion, which is explicit and works under both compilation pipelines
// before
// @ts-expect-error
return response.data;

// after
return response.data as unknown as string;

✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Copilot AI and others added 2 commits February 24, 2026 12:56
Co-authored-by: andrey-helldar <10347617+andrey-helldar@users.noreply.github.com>
Co-authored-by: andrey-helldar <10347617+andrey-helldar@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix existing tests Fix build: replace unused @ts-expect-error with explicit type assertion Feb 24, 2026
Copilot AI requested a review from andrey-helldar February 24, 2026 12:59
@andrey-helldar andrey-helldar marked this pull request as ready for review February 24, 2026 13:07
@andrey-helldar andrey-helldar merged commit 66d6262 into main Feb 24, 2026
7 checks passed
@andrey-helldar andrey-helldar deleted the copilot/fix-tests-again branch February 24, 2026 13:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants