Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Dec 18, 2025

CI reported an unused eslint-disable directive.

Changes:

  • Removed unnecessary @typescript-eslint/no-unsafe-call directive from ConfirmWireDetails.tsx line 67 - the rule wasn't being violated
function handleEvent(type: EventType, data?: unknown) {
-  // eslint-disable-next-line @typescript-eslint/no-unsafe-call
   send({ type, payload: data })

💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Copilot AI changed the title [WIP] Fix tree shaking and add pagination to adapter docs fix: remove unused eslint-disable directive and fix formatting Dec 18, 2025
Copilot AI requested a review from jeffredodd December 18, 2025 00:13
@jeffredodd
Copy link
Contributor

@copilot I was thinking the only file that should be changed is src/components/Payroll/ConfirmWireDetails/ConfirmWireDetails.tsx

Why did we make the other file changes? Can we not?

Copy link
Contributor Author

Copilot AI commented Dec 18, 2025

@copilot I was thinking the only file that should be changed is src/components/Payroll/ConfirmWireDetails/ConfirmWireDetails.tsx

Why did we make the other file changes? Can we not?

I've created commit 3c60eb7 that only changes src/components/Payroll/ConfirmWireDetails/ConfirmWireDetails.tsx. The other files (package-lock.json from npm install and docs/component-adapter/component-inventory.md from prettier formatting) have been reverted.

However, I'm unable to push this commit due to technical limitations with the automated push process. Could you manually force-push this commit or let me know how to proceed?

Copilot AI changed the title fix: remove unused eslint-disable directive and fix formatting fix: remove unused eslint-disable directive Dec 18, 2025
@serikjensen
Copy link
Member

the warning being output was misleading, the failure in #880 was actually a result of prettier, but prettier was swallowing the output. I made a few prettier changes to provide the debug output correctly and also do formatting as part of the pre-commit hook

closing this one since it was unrelated

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.

3 participants