Skip to content

Commit

Permalink
Fix lint issues (#4545)
Browse files Browse the repository at this point in the history
For some reason, two lint violations show up on my computer when running
`yarn lint:eslint`. I have attempted to remove `node_modules`, remove
all `dist` directories, remove `.eslintcache`, run `yarn cache clear`,
etc. to no avail. I think an ultimate fix here may be to upgrade ESLint
packages but that would be a larger change, so as egregious as it is,
this commit skirts around the issue by adding some ignore lines.
  • Loading branch information
mcmire authored Jul 22, 2024
1 parent 51f7584 commit 0f80f16
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 0 deletions.
2 changes: 2 additions & 0 deletions packages/accounts-controller/src/AccountsController.ts
Original file line number Diff line number Diff line change
Expand Up @@ -892,6 +892,8 @@ export class AccountsController extends BaseController<

const index = Math.max(
keyringAccounts.length + 1,
// ESLint is confused; this is a number.
// eslint-disable-next-line @typescript-eslint/restrict-plus-operands
lastDefaultIndexUsedForKeyringType + 1,
);

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -1411,6 +1411,8 @@ describe('NftDetectionController', () => {
it('should not call addNFt when the request to Nft API call throws', async () => {
const selectedAccount = createMockInternalAccount({ address: '0x3' });
nock(NFT_API_BASE_URL)
// ESLint is confused; this is a string.
// eslint-disable-next-line @typescript-eslint/restrict-template-expressions
.get(`/users/${selectedAccount.address}/tokens`)
.query({
continuation: '',
Expand Down

0 comments on commit 0f80f16

Please sign in to comment.