Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Long names for Str, Num, related functions and modules #1164

Merged
merged 5 commits into from
Mar 9, 2023

Conversation

yannham
Copy link
Member

@yannham yannham commented Mar 7, 2023

Depends on #1156. Closes #1147.

Internally, it would make sense to rename Term::Num and Term::Str as well, but this isn't user-facing, and this PR is already substantial 🙃. Let's postpone this to a later phase.

@github-actions github-actions bot temporarily deployed to pull request March 7, 2023 19:34 Inactive
@github-actions github-actions bot temporarily deployed to pull request March 7, 2023 19:49 Inactive
@dpulls
Copy link

dpulls bot commented Mar 8, 2023

🎉 All dependencies have been resolved !

Updates the contract stdlib to expose the new label primop introduced in
a previous commit. Renames existing function accordingly. Get rid of the
`%tag%` primop, now replaced by `%with_message%`.
Rename the builtin `Num` and `Str` types to `Number` and `String`, and
use long names on a number of related functions and stdlib modules for
consistency. This is a breaking syntax change.
@github-actions github-actions bot temporarily deployed to pull request March 8, 2023 15:39 Inactive
@yannham yannham merged commit 5c665e4 into master Mar 9, 2023
@yannham yannham deleted the task/long-type-names branch March 9, 2023 09:33
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.

The Syntax Bikeshedding Dojo, round 11: builtin types and modules
2 participants