Skip to content

Commit

Permalink
Fix lint issues
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 committed Jul 22, 2024
1 parent cc09e4e commit a874cd6
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 a874cd6

Please sign in to comment.