Skip to content

fix(tests): remove unused imports and FIXME comments in dts test fixtures#12428

Draft
Copilot wants to merge 3 commits intoactions/generate-dts-tests-if-neededfrom
copilot/sub-pr-12426
Draft

fix(tests): remove unused imports and FIXME comments in dts test fixtures#12428
Copilot wants to merge 3 commits intoactions/generate-dts-tests-if-neededfrom
copilot/sub-pr-12426

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Mar 16, 2026

The auto-generated DTS export test fixtures introduced two lint failures: unused type imports and FIXME:-prefixed inline comments violating the no-warning-comments ESLint rule.

  • Removed unused importsPreviewUrlResolver, getStudioEnvironmentVariables, and StudioEnvVariablesOptions were listed in imports but only appeared as string labels in test.todo() calls, never in expectTypeOf<>() assertions
  • Removed FIXME comments — Stripped /* FIXME: no declarations found */ from test.todo() call sites; the todo placeholders themselves are retained

🔒 GitHub Advanced Security automatically protects Copilot coding agent pull requests. You can protect all pull requests by enabling Advanced Security for your repositories. Learn more about Advanced Security.

@vercel
Copy link
Copy Markdown

vercel bot commented Mar 16, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
test-studio Ready Ready Preview, Comment Mar 16, 2026 1:12pm
1 Skipped Deployment
Project Deployment Actions Updated (UTC)
page-building-studio Skipped Skipped Mar 16, 2026 1:12pm

Request Review

Co-authored-by: pedrobonamin <46196328+pedrobonamin@users.noreply.github.com>
@vercel vercel bot temporarily deployed to Preview – page-building-studio March 16, 2026 13:12 Inactive
Copilot AI changed the title [WIP] Fix failing checks in DTS tests fix(tests): remove unused imports and FIXME comments in dts test fixtures Mar 16, 2026
Copilot AI requested a review from pedrobonamin March 16, 2026 13:12
@squiggler-app squiggler-app bot force-pushed the actions/generate-dts-tests-if-needed branch 15 times, most recently from 0705326 to 7c314f6 Compare March 19, 2026 14:57
@squiggler-app squiggler-app bot force-pushed the actions/generate-dts-tests-if-needed branch 16 times, most recently from 1e83249 to 01b8b50 Compare March 30, 2026 09:02
@squiggler-app squiggler-app bot force-pushed the actions/generate-dts-tests-if-needed branch 11 times, most recently from 9245488 to 111f86e Compare April 2, 2026 14:35
@squiggler-app squiggler-app bot force-pushed the actions/generate-dts-tests-if-needed branch 3 times, most recently from adf57bb to 449f40c Compare April 7, 2026 11:53
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