Skip to content

Change total lines badge to tokei.rs in README#221

Closed
kaijchen wants to merge 1 commit intoapache:masterfrom
kaijchen:tokei
Closed

Change total lines badge to tokei.rs in README#221
kaijchen wants to merge 1 commit intoapache:masterfrom
kaijchen:tokei

Conversation

@kaijchen
Copy link
Member

What changes were proposed in this pull request?

Change "total lines" badge url to tokei.rs
And use empty alt message to display nothing on failure.

See https://github.com/XAMPPRocky/tokei#badges

Why are the changes needed?

Current badge sometimes show "total lines: invalid".

Does this PR introduce any user-facing change?

No.

How was this patch tested?

https://github.com/kaijchen/incubator-uniffle/tree/tokei#apache-uniffle-incubating

@kaijchen
Copy link
Member Author

Seems #222 is more stable.

@kaijchen kaijchen changed the title Change url of total lines badge in README Change total lines badge to tokei.rs in README Sep 15, 2022
@kaijchen
Copy link
Member Author

Closing this due to #222.

@kaijchen kaijchen closed this Sep 15, 2022
@kaijchen kaijchen deleted the tokei branch January 18, 2023 16:58
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.

1 participant