Skip to content

Add min width languages card #1953

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

Merged
merged 3 commits into from
Aug 18, 2022

Conversation

qimolin
Copy link
Contributor

@qimolin qimolin commented Aug 17, 2022

No description provided.

@vercel
Copy link

vercel bot commented Aug 17, 2022

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

A member of the Team first needs to authorize it.

@qimolin qimolin changed the title Add min width languages card #1945 Add min width languages card Aug 17, 2022
@qimolin
Copy link
Contributor Author

qimolin commented Aug 17, 2022

Implements #1945

@codecov
Copy link

codecov bot commented Aug 18, 2022

Codecov Report

Merging #1953 (b890da4) into master (bdaaba2) will increase coverage by 0.35%.
The diff coverage is 94.44%.

@@            Coverage Diff             @@
##           master    #1953      +/-   ##
==========================================
+ Coverage   94.70%   95.06%   +0.35%     
==========================================
  Files          22       22              
  Lines         775      770       -5     
  Branches      209      209              
==========================================
- Hits          734      732       -2     
+ Misses         36       32       -4     
- Partials        5        6       +1     
Impacted Files Coverage Δ
src/translations.js 81.81% <ø> (ø)
src/calculateRank.js 97.36% <83.33%> (+5.87%) ⬆️
src/cards/top-languages-card.js 100.00% <100.00%> (ø)
src/fetchers/top-languages-fetcher.js 100.00% <100.00%> (ø)

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

@rickstaa
Copy link
Collaborator

@qimolin Thanks a lot for your contribution! I made some minor changes and merged them into the main branch.

@rickstaa rickstaa merged commit dd5a41d into anuraghazra:master Aug 18, 2022
SousaPedro11 pushed a commit to SousaPedro11/github-readme-stats that referenced this pull request Aug 19, 2022
* feat: add min width languages card (anuraghazra#1953)

* Add min width to languages card

* Add test for min width languages card

* feat: increase lang card min width

Co-authored-by: Rick Staa <[email protected]>

* ci: add top-issues-dashboard action

This commit adds the top issues dashboard action which creates a issue displaying the top issues of the week.

* ci: fix title of top-issues action

Co-authored-by: Qi Mo Lin <[email protected]>
Co-authored-by: Rick Staa <[email protected]>
Repository owner deleted a comment from Anngoodness Sep 9, 2022
Repository owner deleted a comment from Anngoodness Sep 9, 2022
Repository owner deleted a comment from Anngoodness Sep 9, 2022
infinity-plus pushed a commit to infinity-plus/github-readme-stats that referenced this pull request Sep 24, 2022
* Add min width to languages card

* Add test for min width languages card

* feat: increase lang card min width

Co-authored-by: Rick Staa <[email protected]>
necofuryai added a commit to necofuryai/github-readme-stats that referenced this pull request May 19, 2025
* Add min width to languages card

* Add test for min width languages card

* feat: increase lang card min width

Co-authored-by: Rick Staa <[email protected]>
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.

2 participants