Ci formatting and linting#26585
Merged
gambinish merged 1 commit intoFeb 25, 2026
Merged
Conversation
Fix Prettier formatting issue in the import statement for PerpsControllerState and UserHistoryItem types from @metamask/perps-controller. The import was on a single line with an extra space before the comma, which violated Prettier's formatting rules. Co-authored-by: Nick Gambino <gambinish@users.noreply.github.com>
Contributor
|
Cursor Agent can help with this pull request. Just |
Contributor
🔍 Smart E2E Test Selection⏭️ Smart E2E selection skipped - base branch is not main (base: perps/fix/stuck-pending-withdraw-4) All E2E tests pre-selected. |
c2c3d50
into
perps/fix/stuck-pending-withdraw-4
103 of 108 checks passed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Fixes
yarn format:checkby correcting an import statement inuseWithdrawalRequests.test.ts.Other
yarn lint:tscandyarn lintfailures were due to local setup not mirroring CI (missing patches and generated files), which are resolved by running the CI setup script. The only actual code change required was this formatting fix.Note
Low Risk
Formatting-only change in a test file with no production logic impact.
Overview
Fixes CI formatting by changing the
@metamask/perps-controllertype-only import inuseWithdrawalRequests.test.tsfrom a single-line import to a multi-line, lint-compliant form.No runtime or behavioral changes; this is a test-file formatting-only update intended to satisfy
yarn format:check.Written by Cursor Bugbot for commit da201ab. This will update automatically on new commits. Configure here.