Skip to content

Remove for orelse for For#1633

Merged
jacqueswww merged 3 commits into
vyperlang:masterfrom
jacqueswww:remove_for_orelse
Oct 24, 2019
Merged

Remove for orelse for For#1633
jacqueswww merged 3 commits into
vyperlang:masterfrom
jacqueswww:remove_for_orelse

Conversation

@jacqueswww

Copy link
Copy Markdown
Contributor

What I did

We don't support For .. orelse. We should not let it blindly not generate code.

How I did it

How to verify it

Description for the changelog

Cute Animal Picture

^.^

Comment thread vyper/parser/stmt.py
Comment thread vyper/ast.py
@fubuloubu

Copy link
Copy Markdown
Contributor

@jacqueswww this is ready for merge, right?

@jacqueswww

jacqueswww commented Oct 24, 2019 via email

Copy link
Copy Markdown
Contributor Author

@jacqueswww jacqueswww merged commit da6d2a3 into vyperlang:master Oct 24, 2019
@protolambda protolambda mentioned this pull request Jan 20, 2020
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