feat: add String() to rueidis.RedisResult for easy debugging

[Exca-DK]

closes #292

[codecov-commenter]

Codecov Report

Patch coverage: 75.43% and project coverage change: -0.05% ⚠️

Comparison is base (19e115b) 97.61% compared to head (706a520) 97.56%.
Report is 4 commits behind head on main.

❗ Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #353      +/-   ##
==========================================
- Coverage   97.61%   97.56%   -0.05%     
==========================================
  Files          76       76              
  Lines       30467    30524      +57     
==========================================
+ Hits        29739    29780      +41     
- Misses        613      623      +10     
- Partials      115      121       +6     

Files Changed	Coverage Δ
message.go	98.42% <75.43%> (-1.58%)	⬇️

... and 2 files with indirect coverage changes

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[rueian]

Hi @Exca-DK,

Thank you for implementing this. However, I am hesitant about adding the json.Marshaler interface.

People will most likely expect the json.Marshaler should be performant and its output can be restored back to the source struct by using json.Unmarshal.

I think we had better just keep the Stringer interface which is widely accepted as a debugging helper.

[Exca-DK]

You are right, I made it into an internal type, it shouldn't be known to people at all. What do you think?

[rueian]

Hi @Exca-DK,

Could you also add tests for these uncovered lines?

[rueian]

Thanks @Exca-DK!