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

rangeFormatting request including context clause raises AssertionError #1073

Closed
stephe-ada-guru opened this issue Nov 8, 2022 · 2 comments
Closed

rangeFormatting request including context clause raises AssertionError #1073

stephe-ada-guru opened this issue Nov 8, 2022 · 2 comments
Assignees
@joaopsazevedo

Comments

@stephe-ada-guru
Copy link

The attached test.json.txt shows an error; the attached range_formatting_context.diff (against libadalang_tools) fixes it.
test.json.txt
range_formatting_context.diff.txt
@stephe-ada-guru
Copy link
Author

There are additional problems with the response once the assertion_error is fixed; I'll submit another issue.

@stephe-ada-guru stephe-ada-guru mentioned this issue Nov 8, 2022
Closed
@joaopsazevedo joaopsazevedo self-assigned this Dec 14, 2022
@joaopsazevedo joaopsazevedo mentioned this issue Jan 3, 2023
Merged
joaopsazevedo added a commit that referenced this issue Jan 3, 2023
@joaopsazevedo
W103-015 Add tests that range formats the whole file
3ae3674 
fix #1073
fix #1074
fix #1086
joaopsazevedo added a commit that referenced this issue Jan 4, 2023
@joaopsazevedo
W103-015 Add tests that range formats the whole file
4f7a401 
fix #1073
fix #1074
fix #1086
joaopsazevedo added a commit that referenced this issue Jan 5, 2023
@joaopsazevedo
W103-015 Add tests that range formats the whole file
244601e 
fix #1073
fix #1074
fix #1086
@joaopsazevedo
Copy link
Contributor

This issue is now fixed in the the edge branch.
Thank you for reporting and for the reproducer.

AnthonyLeonardoGracio pushed a commit that referenced this issue Jan 23, 2023
@joaopsazevedo
W103-015 Add tests that range formats the whole file
ff373ca 
fix #1073
fix #1074
fix #1086
AnthonyLeonardoGracio pushed a commit that referenced this issue Jan 23, 2023
@joaopsazevedo @AnthonyLeonardoGracio
W103-015 Add tests that range formats the whole file
0ce1dfc 
fix #1073
fix #1074
fix #1086
AnthonyLeonardoGracio pushed a commit that referenced this issue Jan 23, 2023
@joaopsazevedo @AnthonyLeonardoGracio
W103-015 Add tests that range formats the whole file
5280cd7 
fix #1073
fix #1074
fix #1086
godunko pushed a commit that referenced this issue Jan 24, 2023
@joaopsazevedo @godunko
W103-015 Add tests that range formats the whole file
83015b2 
fix #1073
fix #1074
fix #1086
godunko pushed a commit that referenced this issue Jan 24, 2023
@joaopsazevedo @godunko
W103-015 Add tests that range formats the whole file
48cf5aa 
fix #1073
fix #1074
fix #1086
godunko pushed a commit that referenced this issue Jan 24, 2023
@joaopsazevedo @godunko
W103-015 Add tests that range formats the whole file
61b0921 
fix #1073
fix #1074
fix #1086
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@joaopsazevedo joaopsazevedo

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@stephe-ada-guru @joaopsazevedo