Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ESLint: Update rules #107

Open
wants to merge 5 commits into
base: master
Choose a base branch
from
Open

Conversation

juanchoperezj
Copy link

What does this do?

Add new linting rules.

Why did you do this?

The objective is to reduce code review time by preventing changes that ESLint can address during development.
Card

Who/what does this impact?

Few existing LOCs were affected by this.

How did you test this?

Zero issues by running pnpm run lint.

Copy link

github-actions bot commented Dec 9, 2024

💯 Test Coverage

Lines Statements Branches Functions
Coverage: 52%
52.03% (268/515) 44.79% (99/221) 37.82% (73/193)

😎 Tests Results

Tests Skipped Failures Errors Time
66 0 💤 0 ❌ 0 🔥 28.016s ⏱️
👀 Tests Details • (52%)
File% Stmts% Branch% Funcs% LinesUncovered Line #s
All files52.0344.7937.8252.76 
api/common82.65062.586.04 
   utils.tsx85.294554.5490.3214–20
api/posts010000 
   use-post.ts01000010–13
app/(app)0000 
   settings.tsx000013–18
ui77.7261.5369.2377.41 
   button.tsx10087.5100100126–132

Copy link
Collaborator

@fernandatoledo fernandatoledo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👏

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants