Skip to content

Conversation

@heyolaniran
Copy link
Contributor

Hello @willcl-ark , in this PR i made some addition to the presence of typescript in the list of languages used in Bitcoin Core Repository.

In the Project stats, Tokei considers all .ts files as typescript files. In the case of Bitcoin Core, .ts are related to translation files.

Thank you for having time to review my suggestion and feel free to leave me any insights for improvements.

In the Project stats, Tokei considers all .ts files as typescript files. In the case of Bitcoin Core, .ts are related to translation files.
@appletorsh19-stack
Copy link

#57

@appletorsh19-stack
Copy link

#97

@willcl-ark
Copy link
Collaborator

Hello @willcl-ark , in this PR i made some addition to the presence of typescript in the list of languages used in Bitcoin Core Repository.

In the Project stats, Tokei considers all .ts files as typescript files. In the case of Bitcoin Core, .ts are related to translation files.

Thank you for having time to review my suggestion and feel free to leave me any insights for improvements.

Hey @heyolaniran, thanskf for the PR.

I think perhaps the clearer approach here would be to manually remove the Typescript line altogether (rather than add a note), and next time it's re-calculated on a version bump we can use tokei -e *.ts I beileve to exclude that filetype.

I don't think highlighting translation chars is particularly useful in any way.

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