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

Remove module-level order restrictions #2057

Conversation

iamdefinitelyahuman
Copy link
Contributor

What I did

Remove restrictions on ordering of elements within the module-level namespace of a contract.

The only restriction this does not remove is that functions may only call upward - that logic is fairly intertwined with global_context and will take some work to eliminate.

How I did it

  • Removed some checks in global_context
  • Removed failing tests
  • Checked the documentation - this behavior was never documented so nothing needs to be changed there

How to verify it

Run tests.

Cute Animal Picture

image

@iamdefinitelyahuman iamdefinitelyahuman merged commit c104fc9 into vyperlang:master Jun 27, 2020
@iamdefinitelyahuman iamdefinitelyahuman deleted the remove-order-restrictions branch June 27, 2020 13:30
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.

None yet

2 participants