Skip to content
This repository has been archived by the owner on May 11, 2024. It is now read-only.

chore(prover): replace emoji in log message #351

Merged
merged 1 commit into from
Aug 7, 2023

Conversation

Nakochi-crypto
Copy link
Contributor

This pull request just replaces 🤷‍♂️ with 🤷 in the log messages.

Depending on the environment, 🤷‍♂️ is not displayed properly because it consists of 4 code points.

Code point Character Name
U+1F937 🤷 Shrug
U+200D Zero Width Joiner (ZWJ)
U+2642 Male Sign
U+FE0F ◌️ Variation Selector-16 (VS16)

On iTerm2 on macOS, it is displayed as 🤷\u200d♂️.

From the perspective of gender equality, it would be better to use the neutral one.

@davidtaikocha davidtaikocha enabled auto-merge (squash) August 7, 2023 01:59
@codecov
Copy link

codecov bot commented Aug 7, 2023

Codecov Report

Merging #351 (ba68d58) into main (19ada49) will increase coverage by 0.36%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main     #351      +/-   ##
==========================================
+ Coverage   51.71%   52.07%   +0.36%     
==========================================
  Files          36       36              
  Lines        4438     4438              
==========================================
+ Hits         2295     2311      +16     
+ Misses       1880     1863      -17     
- Partials      263      264       +1     
Files Changed Coverage Δ
prover/proof_submitter/util.go 50.00% <100.00%> (ø)

... and 2 files with indirect coverage changes

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@davidtaikocha davidtaikocha merged commit cc1c4a7 into taikoxyz:main Aug 7, 2023
@davidtaikocha
Copy link
Member

Merged, thanks for the contribution!

@Nakochi-crypto Nakochi-crypto deleted the replace-emoji branch August 7, 2023 07:28
@Nakochi-crypto
Copy link
Contributor Author

I heard that those whose pull requests are merged will receive a GitPOAP. Am I eligible?

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.

2 participants