Skip to content

fix: Remove unused imports and superfluous arguments in tests - #113

Merged
hzhangxyz merged 1 commit into
mainfrom
dev/superfluous-trailing-arguments
Mar 16, 2026
Merged

fix: Remove unused imports and superfluous arguments in tests#113
hzhangxyz merged 1 commit into
mainfrom
dev/superfluous-trailing-arguments

Conversation

@hzhangxyz

Copy link
Copy Markdown
Member
  • test_chain.ts: Remove unused Idea import
  • test_dump.ts: Remove superfluous addr argument from main() calls

- test_chain.ts: Remove unused Idea import
- test_dump.ts: Remove superfluous addr argument from main() calls
Copilot AI review requested due to automatic review settings March 16, 2026 04:20

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Cleans up test code by removing unused imports and aligning test calls with the current main() function signatures (notably for dump).

Changes:

  • Update tests/test_dump.ts to call main(sequelize) (remove unused addr argument).
  • Remove unused Idea import from tests/test_chain.ts.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
tests/test_dump.ts Updates test invocations to match dump.main(sequelize) signature.
tests/test_chain.ts Removes unused ORM import to reduce noise in tests.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

You can also share your feedback on Copilot code review. Take the survey.

@hzhangxyz
hzhangxyz merged commit b8da4cd into main Mar 16, 2026
32 checks passed
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