Skip to content

Commit

Permalink
Add no-console ESLint rule
Browse files Browse the repository at this point in the history
  • Loading branch information
wcalderipe committed May 3, 2024
1 parent 44dce44 commit 5f57559
Show file tree
Hide file tree
Showing 7 changed files with 8 additions and 6 deletions.
1 change: 1 addition & 0 deletions .eslintrc.json
Original file line number Diff line number Diff line change
Expand Up @@ -6,6 +6,7 @@
{
"files": ["*.ts", "*.tsx", "*.js", "*.jsx"],
"rules": {
"no-console": "error",
"@nx/enforce-module-boundaries": [
"error",
{
Expand Down
1 change: 1 addition & 0 deletions apps/armory/src/shared/module/persistence/seed.ts
Original file line number Diff line number Diff line change
Expand Up @@ -47,6 +47,7 @@ main()
await prisma.$disconnect()
})
.catch(async (e) => {
// eslint-disable-next-line no-console
console.error(e)
await prisma.$disconnect()
process.exit(1)
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -135,10 +135,7 @@ const PlaygroundEditor = () => {
<div className="pt-4">
<h3 className="pb-4">Examples</h3>
<div className="flex gap-5 ">
<NarButton
label="ERC-20"
onClick={async () => setCodeEditor(JSON.stringify(await erc20(), null, 2))}
/>
<NarButton label="ERC-20" onClick={async () => setCodeEditor(JSON.stringify(await erc20(), null, 2))} />
<NarButton
label="Spending limits"
onClick={async () => setCodeEditor(JSON.stringify(await spendingLimits(), null, 2))}
Expand Down
2 changes: 2 additions & 0 deletions apps/policy-engine/src/cli/command/provision.command.ts
Original file line number Diff line number Diff line change
@@ -1,3 +1,5 @@
/* eslint-disable no-console */

import { ConfigService } from '@narval/config-module'
import { Command, CommandRunner } from 'nest-commander'
import { EngineService } from '../../engine/core/service/engine.service'
Expand Down
2 changes: 2 additions & 0 deletions apps/vault/src/cli/command/provision.command.ts
Original file line number Diff line number Diff line change
@@ -1,3 +1,5 @@
/* eslint-disable no-console */

import { ConfigService } from '@nestjs/config'
import { Command, CommandRunner } from 'nest-commander'
import { Config } from '../../main.config'
Expand Down
1 change: 0 additions & 1 deletion apps/vault/src/vault/vault.service.ts
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,6 @@ export class VaultService {
constructor(private signingService: SigningService) {}

async sign(): Promise<EvaluationResponse | null> {
console.log('Signing Called')
return null
}
}
2 changes: 1 addition & 1 deletion packages/signature/.eslintrc.json
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
{
"extends": ["../../.eslintrc.json"],
"ignorePatterns": ["!**/*"],
"ignorePatterns": ["!**/*", "dist/**/*"],
"overrides": [
{
"files": ["*.ts", "*.tsx", "*.js", "*.jsx"],
Expand Down

0 comments on commit 5f57559

Please sign in to comment.