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

@as value is not checked properly #3629

Closed
yegor256 opened this issue Dec 9, 2024 · 0 comments · Fixed by #3633
Closed

@as value is not checked properly #3629

yegor256 opened this issue Dec 9, 2024 · 0 comments · Fixed by #3633
Assignees
@maxonfjvipon
Labels
bug help wanted

Comments

@yegor256
Copy link
Member

yegor256 commented Dec 9, 2024

I'm adding this YAML pack to eo-parser/src/test/resources/org/eolang/typos and the test doesn't pass:

line: 4
input: |
  # No comments
  [] > foo
    bar
      42:-1

Here, -1 is definitely a wrong value of the @as attribute. I believe, parser must fail in this case. Also, it must fail with +1.
@maxonfjvipon maxonfjvipon self-assigned this Dec 10, 2024
maxonfjvipon added a commit to maxonfjvipon/eo that referenced this issue Dec 10, 2024
@maxonfjvipon
fix(objectionary#3629): fixed broken @as binding
cd7587e
@maxonfjvipon maxonfjvipon mentioned this issue Dec 10, 2024
Merged
maxonfjvipon added a commit to maxonfjvipon/eo that referenced this issue Dec 10, 2024
@maxonfjvipon
fix(objectionary#3629): listener errors
38b29a7
maxonfjvipon added a commit to maxonfjvipon/eo that referenced this issue Dec 10, 2024
@maxonfjvipon
fix(objectionary#3629): grammar
dfab3a6
yegor256 added a commit that referenced this issue Dec 11, 2024
@yegor256
Merge pull request #3633 from maxonfjvipon/fix/#3629/broken-as-binding
6aa9ba2 
fix(#3629): fixed broken `@as` binding
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@maxonfjvipon maxonfjvipon

Labels
bug help wanted
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

fix(#3629): fixed broken @as binding maxonfjvipon/eo
2 participants
@yegor256 @maxonfjvipon