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.

Sign up for GitHub

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

Jump to bottom

[core] Fix GitHub language detection #27298

Merged
merged 3 commits into from
Jul 17, 2021
Merged

[core] Fix GitHub language detection #27298

merged 3 commits into from
Jul 17, 2021

Conversation

oliviertassinari
Copy link
Member

@oliviertassinari oliviertassinari commented Jul 15, 2021 •
edited
Loading

I was working on #15984 to monitor our progress on the issue:

Capture d’écran 2021-07-15 à 15 56 29

When I realize that we broke the language stats. Before

89.82%  JavaScript
10.17%  TypeScript
0.01%   HTML

After

64.52%  JavaScript
35.44%  TypeScript
0.04%   HTML

The change is based on github-linguist/linguist#5439 (comment).

@oliviertassinari oliviertassinari added typescript core Infrastructure work going on behind the scenes labels Jul 15, 2021
@mui-pr-bot
Copy link

mui-pr-bot commented Jul 15, 2021 •
edited
Loading

No bundle size changes (experimental)

Generated by 🚫 dangerJS against 1fc8809

eps1lon
eps1lon reviewed Jul 15, 2021
View reviewed changes
.gitattributes Show resolved Hide resolved
.gitattributes Show resolved Hide resolved
@oliviertassinari oliviertassinari merged commit f50960a into mui:next Jul 17, 2021
@oliviertassinari oliviertassinari deleted the github-source-label branch July 17, 2021 14:33
@oliviertassinari
Copy link
Member Author

No surprises

Capture d’écran 2021-07-17 à 21 10 18

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@eps1lon eps1lon eps1lon approved these changes

Assignees
No one assigned
Labels
core Infrastructure work going on behind the scenes typescript
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@oliviertassinari @mui-pr-bot @eps1lon