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 integration #2885

Merged
merged 8 commits into from
Jul 3, 2023
Merged

Eslint integration #2885

merged 8 commits into from
Jul 3, 2023

Conversation

qwerty541
Copy link
Collaborator

Related to #2882 (comment)

@qwerty541 qwerty541 requested a review from rickstaa June 27, 2023 11:14
@vercel
Copy link

vercel bot commented Jun 27, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated (UTC)
github-readme-stats ✅ Ready (Inspect) Visit Preview Jul 2, 2023 9:32pm

@github-actions github-actions bot added dependencies Pull requests that update a dependency file ranks Feature, Bug fix, improvement related to ranking system. labels Jun 27, 2023
@codecov
Copy link

codecov bot commented Jun 27, 2023

Codecov Report

Patch coverage: 100.00% and no project coverage change.

Comparison is base (de4efa9) 97.61% compared to head (81c0def) 97.61%.

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #2885   +/-   ##
=======================================
  Coverage   97.61%   97.61%           
=======================================
  Files          24       24           
  Lines        5156     5159    +3     
  Branches      460      460           
=======================================
+ Hits         5033     5036    +3     
  Misses        122      122           
  Partials        1        1           
Impacted Files Coverage Δ
src/calculateRank.js 100.00% <100.00%> (ø)
src/getStyles.js 73.91% <100.00%> (+0.38%) ⬆️

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

@qwerty541 qwerty541 linked an issue Jun 27, 2023 that may be closed by this pull request
@github-actions github-actions bot added the ci CI related features. label Jun 27, 2023
@qwerty541 qwerty541 marked this pull request as ready for review June 27, 2023 13:54
Copy link
Collaborator Author

@qwerty541 qwerty541 left a comment

Choose a reason for hiding this comment

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

@rickstaa This is basic integration of eslint. As you see most of rules inside eslintrc.json under comments, enabled only object-shorthand and default. We can gradually uncomment rules and improve code quality. I also added checking lints on continuous integration and pre commit hook.

src/getStyles.js Outdated Show resolved Hide resolved
Copy link
Collaborator

@rickstaa rickstaa left a comment

Choose a reason for hiding this comment

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

See my comments. Appart from these two, I think this PR is good to go.

Copy link
Collaborator

@rickstaa rickstaa left a comment

Choose a reason for hiding this comment

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

See my suggestion

tests/renderWakatimeCard.test.js Show resolved Hide resolved
@rickstaa rickstaa merged commit 6c92fe9 into master Jul 3, 2023
8 checks passed
@rickstaa rickstaa deleted the eslint_integration branch July 3, 2023 06:54
J00MZ pushed a commit to J00MZ/github-readme-stats that referenced this pull request Jul 23, 2023
* eslint integration

* ci & pre commit

* dev

* dev

* dev
devantler pushed a commit to devantler/github-readme-stats that referenced this pull request Sep 24, 2023
* eslint integration

* ci & pre commit

* dev

* dev

* dev
setdebarr pushed a commit to setdebarr/github-readme-stats that referenced this pull request Jan 12, 2024
* eslint integration

* ci & pre commit

* dev

* dev

* dev
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ci CI related features. dependencies Pull requests that update a dependency file ranks Feature, Bug fix, improvement related to ranking system.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Eslint integration
3 participants