Skip to content
This repository has been archived by the owner on Oct 24, 2023. It is now read-only.

style: override linguist's default language detection #3100

Merged
merged 1 commit into from Apr 20, 2020

Conversation

mboersma
Copy link
Member

Reason for Change:
Overrides the default language detection for github-linguist so Go template files aren't identified as Raku or Perl.

Before:

# github-linguist --breakdown
91.54%  Go
2.79%   Raku
2.61%   Shell
1.99%   PowerShell
0.90%   Perl
0.14%   Makefile
0.03%   Python
0.00%   Batchfile

After:

# github-linguist --breakdown
95.22%  Go
2.61%   Shell
1.99%   PowerShell
0.14%   Makefile
0.03%   Python
0.00%   Batchfile

Issue Fixed:

Requirements:

Notes:
I realize this is entirely trivial, but it's the kind of thing that bothers me.

@codecov
Copy link

codecov bot commented Apr 20, 2020

Codecov Report

Merging #3100 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #3100   +/-   ##
=======================================
  Coverage   70.62%   70.62%           
=======================================
  Files         147      147           
  Lines       25407    25407           
=======================================
  Hits        17944    17944           
  Misses       6341     6341           
  Partials     1122     1122           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update a1a1adb...742ed0c. Read the comment docs.

Copy link
Member

@jackfrancis jackfrancis left a comment

Choose a reason for hiding this comment

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

/lgtm

@mboersma mboersma merged commit 683de7a into Azure:master Apr 20, 2020
@mboersma mboersma deleted the override-linguist branch April 20, 2020 18:39
alexeldeib pushed a commit to alexeldeib/aks-engine that referenced this pull request Apr 21, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants