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

Support Clarity language changes in Stacks 2.1 #465

Closed
obycode opened this issue Jan 26, 2022 · 4 comments
Closed

Support Clarity language changes in Stacks 2.1 #465

obycode opened this issue Jan 26, 2022 · 4 comments
Assignees
Labels
functional enhancement New feature or request

Comments

@obycode
Copy link
Member

obycode commented Jan 26, 2022

With the switch to 2.1, we have the opportunity to make some breaking changes to the language. It would make sense to put together a formal specification of the language, including some breaking changes (e.g. limitations on symbol names like this<is>con/fus!ing*). It would be nice to have a BNF specification (or something else). The new parser (see hirosystems/clarity-repl#74) will implement these changes.

The task to define the language specification has been moved to stacks-network/stacks-core#3052. This issue should track that issue and make the corresponding changes in the REPL parser.

@obycode obycode added the functional enhancement New feature or request label Jan 26, 2022
@obycode obycode self-assigned this Jan 26, 2022
@fariedt
Copy link

fariedt commented Feb 3, 2022

I don't have a BNF, but I came up with this using instaparse a few months ago, and it works for pretty much everything I've thrown at it. I had to strip comments, though.

@LNow
Copy link
Contributor

LNow commented Feb 17, 2022

@obycode I think this discussion should be started in stacks-blockchain repo, as this is where new language functions are added. For sure we need some form of specification to eliminate weirdness I described here: stacks-network/stacks-core#2696 And unfortunately repl will should support both old and new specification.

@fariedt this looks awesome!!!

@obycode
Copy link
Member Author

obycode commented Feb 17, 2022

Agreed. Sounds good to me. I opened stacks-network/stacks-core#3052 and will modify this issue to track that issue and make the corresponding changes in the REPL.

@obycode obycode changed the title Formal specification of Clarity for Stacks 2.1 Support Clarity language changes in Stacks 2.1 Feb 17, 2022
@lgalabru lgalabru transferred this issue from hirosystems/chainhook Jul 12, 2022
@lgalabru
Copy link
Contributor

lgalabru commented Oct 6, 2022

@obycode now that everything lives on the stacks-blockchain side, can we close this issue?

@obycode obycode closed this as completed Oct 6, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
functional enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

4 participants