onflow / cadence

Cadence, the resource-oriented smart contract programming language 🏃‍♂️
https://cadence-lang.org
Apache License 2.0
534 stars 138 forks source link

Introduce StructStringer #3594

Closed RZhang05 closed 3 weeks ago

RZhang05 commented 2 months ago

Working towards #3579

Description

Implement https://github.com/onflow/flips/pull/294

Add a simple StructStringer struct interface. It is intended in the future that all types which implement StructStringer can be used in string templates.


SupunS commented 1 month ago

@RZhang05 is this ready to be reviewed? or are there any pending changes?

RZhang05 commented 1 month ago

@RZhang05 is this ready to be reviewed? or are there any pending changes?

It is ready, note that there are some questions left in the code marked with TODO however.

turbolent commented 1 month ago

@SupunS Was this closed by accident?

SupunS commented 1 month ago

oh, my bad. I was cleaning up some of my old branches, and must have accidentally deleted the base branch! Thought it was protected. Will revert it.

Restored the and reopened. Sorry about that 🙇

turbolent commented 1 month ago

No worries at all, I was just not sure about the feature branches 👍