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

separate library from other top level declarations #7

Merged
merged 1 commit into from
May 31, 2021

Conversation

leviathanbeak
Copy link
Contributor

since script, contract and predicate don't take a name after declaration I needed to separate them from library syntax pattern

Verified

This commit was signed with the committer’s verified signature.
Copy link
Contributor

@adlerjohn adlerjohn left a comment

Choose a reason for hiding this comment

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

This looks fine for now. @sezna shouldn't libraries by a top-level thing eventually?

@sezna
Copy link

sezna commented May 28, 2021

I'm not 100% sure what you mean, @adlerjohn . They already are a top-level thing, they just have a slightly different syntax as they require a name. e.g.:

library foo;

pub fn bar() {}

@adlerjohn
Copy link
Contributor

They already are a top-level thing

Ah you're right I misread the code. Carry on 👍

@leviathanbeak leviathanbeak merged commit 7405f67 into master May 31, 2021
@leviathanbeak leviathanbeak deleted the update-syntaxes branch May 31, 2021 10:07
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