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

fix: fix bad GitHub Linguist link #1277

Merged
merged 1 commit into from
Nov 6, 2022
Merged

fix: fix bad GitHub Linguist link #1277

merged 1 commit into from
Nov 6, 2022

Conversation

mastondzn
Copy link
Contributor

This fixes a bad link introduced in #1098

Discord_1rZYYIiFEY

@lowlighter lowlighter merged commit 0f720e8 into lowlighter:master Nov 6, 2022
@lowlighter
Copy link
Owner

Nice catch !
Thanks for your contribution !

@mastondzn mastondzn deleted the fix-linguist-link branch November 6, 2022 03:32
@@ -314,7 +314,7 @@ <h2>
<p>
<b>{{ user }}</b> has used {{ languages.length }} different language{{ format("plural", languages.length) }} for a total of {{ format("number", languages.total) }} byte{{ format("plural", languages.total) }}.
<small class="footnote">
Note that these numbers are based on results produced by <a href="github.com/github/linguist">GitHub linguist</a> from repositories owned by <b>{{ user }}</b> and that they may not be entirely accurate.
Copy link
Contributor

Choose a reason for hiding this comment

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

Metrics uses LinguistJS now, should this be changed to link to that instead?

Copy link
Owner

Choose a reason for hiding this comment

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

Yeah we should update it 👍

@github-actions github-actions bot locked and limited conversation to collaborators Nov 27, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants