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.

Sign up for GitHub

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

Jump to bottom

Fixes #4361 wrong error message: unexpected token: end #4424

Merged
merged 1 commit into from May 17, 2017
Merged

Fixes #4361 wrong error message: unexpected token: end #4424

merged 1 commit into from May 17, 2017

Conversation

akzhan
Copy link
Contributor

@akzhan akzhan commented May 17, 2017

Replace "expecting when, else or end" with "expecting when or else".

Spec example added.

@akzhan
Fixes crystal-lang#4361 wrong error message: unexpected token: end
755f397 
Replace "expecting when, else or end" with "expecting when or else".
@akzhan
Copy link
Contributor Author

akzhan commented May 17, 2017

Build failed on macOS:xcode8.2 due to Travis issue:

Error: Failed to download resource "llvm"
Download failed: https://homebrew.bintray.com/bottles/llvm-4.0.0_1.sierra.bottle.tar.gz

@RX14
Copy link
Contributor

RX14 commented May 17, 2017

Travis builds have been intermittently acting up on osx for weeks.

@akzhan akzhan closed this May 17, 2017
@akzhan
Copy link
Contributor Author

akzhan commented May 17, 2017

Reopened to retry the build.

@akzhan akzhan reopened this May 17, 2017
@bcardiff bcardiff added this to the Next milestone May 17, 2017
@mverzilli mverzilli merged commit fe78661 into crystal-lang:master May 17, 2017
@mverzilli
Copy link

Thanks @akzhan!

@akzhan akzhan deleted the fix-4361-unexpected-token-end-expecting-when-else-or-end branch May 18, 2017 17:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@bcardiff bcardiff bcardiff approved these changes

Assignees
No one assigned
Labels
kind:bug topic:compiler:parser
Projects
None yet
Milestone
0.23.0
Development

Successfully merging this pull request may close these issues.

None yet
4 participants
@akzhan @RX14 @mverzilli @bcardiff