Skip to content

Fix Download Stats By Language #93

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 2 commits into from
Jun 4, 2019
Merged

Conversation

sommersoft
Copy link
Collaborator

Just needed to tweak it a little, since now both the stable and pre-releases will have multiple languages.

Download stats by language:

| Board            | 4.0.1  | 4.0.0-rc.3  |
| -----------------|--------|-------------|
| ID               |   7    |      0      |
| de_DE            |   28   |      4      |
| en_US            |  1914  |     533     |
| en_x_pirate      |   31   |      6      |
| es               |   47   |      5      |
| fil              |   2    |      0      |
| fr               |   55   |     10      |
| it_IT            |   7    |      2      |
| pl               |   8    |      1      |
| pt_BR            |   10   |      5      |
| zh_Latn_pinyin   |   5    |      1      |
| -----------------|--------|-------------|
|            Total |  2114  |     567     |
| -----------------|--------|-------------|

Relevant code is in lines 267 thru 297. I keep PEP8ing in increments, fully knowing that its going to muddy up the PRs. Apologies.

@sommersoft sommersoft requested review from tannewt and kattni June 4, 2019 05:12
Copy link
Contributor

@kattni kattni left a comment

Choose a reason for hiding this comment

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

Thanks for the fix and cleanup!

@kattni kattni merged commit c75c29b into adafruit:master Jun 4, 2019
@tannewt
Copy link
Member

tannewt commented Jun 4, 2019

Thank you!

@sommersoft sommersoft deleted the fix_by_lang branch March 15, 2020 22:40
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.

3 participants