Skip to content

Rename "cbor_hex" in the configuration to "inline"#163

Merged
tweag-ev-ak merged 1 commit intomainfrom
confm-up
Mar 16, 2026
Merged

Rename "cbor_hex" in the configuration to "inline"#163
tweag-ev-ak merged 1 commit intomainfrom
confm-up

Conversation

@adithyaov
Copy link
Member

No description provided.

Copy link
Collaborator

@tweag-ev-ak tweag-ev-ak left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Comment on lines +143 to +149
- name: "Always failing script"
hash: "793f8c8cffba081b2a56462fc219cc8fe652d6a338b62c7b134876e7"
source:
inline:
type: "PlutusScriptV2"
description: "Always failing script"
cborHex: "4e4d01000033222220051200120011"
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm looking and at this and wonder, what if we use description as "names" for our scripts?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I had this though earlier but it only is useful for inline scripts. If the scripts are in a file then the barrier to change the name is very high.

Copy link
Contributor

@axman6 axman6 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@tweag-ev-ak tweag-ev-ak merged commit c2e838d into main Mar 16, 2026
2 checks passed
@tweag-ev-ak tweag-ev-ak deleted the confm-up branch March 16, 2026 12:22
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