Skip to content
This repository has been archived by the owner on Sep 18, 2024. It is now read-only.

[Task]: Add no-console to Eslint Rules #181

Closed
btabaska opened this issue Aug 27, 2024 · 1 comment · Fixed by #182
Closed

[Task]: Add no-console to Eslint Rules #181

btabaska opened this issue Aug 27, 2024 · 1 comment · Fixed by #182

Comments

@btabaska
Copy link
Collaborator

Summary

ESlint should catch if we have log statements in linting

Acceptance criteria

/* eslint no-console: "error" */ added to ESlint config
This was referenced Aug 27, 2024
Merged
Merged
btabaska added a commit that referenced this issue Aug 28, 2024
@btabaska
[Issue #181]Add no console log to eslint rules (#182)
7f7103d 
## Summary
Fixes #181 

### Time to review: __1 mins__

## Changes proposed
> Added new rule to eslint to prevent leaving behind console log
statements

## Context for reviewers
> normal expected rule in eslint configurations
@acouch
Copy link
Member

acouch commented Sep 17, 2024

Issue migrated to HHS#2037

@acouch acouch mentioned this issue Sep 17, 2024
Closed
@acouch acouch removed the Migrated label Sep 17, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[Issue #181]Add no console log to eslint rules navapbc/simpler-grants-gov
2 participants
@acouch @btabaska