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

Feature/grs 1955 change commonjs imports #1995

Merged
merged 8 commits into from
Sep 24, 2022
Merged

Feature/grs 1955 change commonjs imports #1995

merged 8 commits into from
Sep 24, 2022

Conversation

rsk2
Copy link
Contributor

@rsk2 rsk2 commented Sep 3, 2022

No description provided.

@vercel
Copy link

vercel bot commented Sep 3, 2022

@rsk2 is attempting to deploy a commit to the github readme stats Team on Vercel.

A member of the Team first needs to authorize it.

@github-actions github-actions bot added card-i18n Card text translations. themes Feature, Enhancement, Fixes related to themes. labels Sep 3, 2022
This commit makes sure jest is set-up to support es6. It also fixes
several test errors and sorts the imports.
@rsk2 rsk2 marked this pull request as ready for review September 18, 2022 08:29
@rickstaa rickstaa self-requested a review September 20, 2022 06:54
@rickstaa
Copy link
Collaborator

@rsk2 I checked this PR, and something weird is happening. It looks like not all changes in https://github.com/rsk2/github-readme-stats/pull/1/files are found in this PR. I think this is because merge conflicts in 275ec70 were not correctly dealt with.

image

Still is

image

Which is incorrect and will throw an error.

@rickstaa
Copy link
Collaborator

This commit makes sure node 16 is used in the github actions.
@codecov
Copy link

codecov bot commented Sep 24, 2022

Codecov Report

Base: 95.20% // Head: 94.66% // Decreases project coverage by -0.53% ⚠️

Coverage data is based on head (de42345) compared to base (4073512).
Patch coverage: 100.00% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1995      +/-   ##
==========================================
- Coverage   95.20%   94.66%   -0.54%     
==========================================
  Files          22       22              
  Lines         792      712      -80     
  Branches      219      219              
==========================================
- Hits          754      674      -80     
  Misses         33       33              
  Partials        5        5              
Impacted Files Coverage Δ
src/calculateRank.js 97.29% <ø> (-0.08%) ⬇️
src/cards/repo-card.js 100.00% <ø> (ø)
src/cards/stats-card.js 100.00% <ø> (ø)
src/cards/top-languages-card.js 100.00% <ø> (ø)
src/cards/wakatime-card.js 98.61% <ø> (-0.16%) ⬇️
src/common/Card.js 95.08% <ø> (-0.24%) ⬇️
src/common/I18n.js 80.00% <ø> (-1.82%) ⬇️
src/common/blacklist.js 100.00% <ø> (ø)
src/common/createProgressNode.js 100.00% <ø> (ø)
src/common/icons.js 100.00% <ø> (ø)
... and 12 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

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

@rickstaa
Copy link
Collaborator

All problems are fixed now. @rsk2 Thanks a lot for your contribution! 🚀 It greatly improves the codebase.

@rickstaa rickstaa merged commit 107f7ca into anuraghazra:master Sep 24, 2022
@rsk2
Copy link
Contributor Author

rsk2 commented Sep 24, 2022

Thanks for your help and guidance @rickstaa!

infinity-plus pushed a commit to infinity-plus/github-readme-stats that referenced this pull request Sep 24, 2022
* GRS-1955: Using ES6 import/export in src files

* GRS-1955: Using ES6 import/export in test files

* GRS-1955: Using ES6 import/export in themes index.js

* GRS-1955: Readding blank line at end of top-languages-card.js

* feat: fix test es6 import errors

This commit makes sure jest is set-up to support es6. It also fixes
several test errors and sorts the imports.

* test: update test node version

This commit makes sure node 16 is used in the github actions.

* refactor: run prettier

Co-authored-by: rickstaa <[email protected]>
devantler pushed a commit to devantler/github-readme-stats that referenced this pull request Sep 26, 2022
* GRS-1955: Using ES6 import/export in src files

* GRS-1955: Using ES6 import/export in test files

* GRS-1955: Using ES6 import/export in themes index.js

* GRS-1955: Readding blank line at end of top-languages-card.js

* feat: fix test es6 import errors

This commit makes sure jest is set-up to support es6. It also fixes
several test errors and sorts the imports.

* test: update test node version

This commit makes sure node 16 is used in the github actions.

* refactor: run prettier

Co-authored-by: rickstaa <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
card-i18n Card text translations. themes Feature, Enhancement, Fixes related to themes.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants