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

Fewer braces are not fully supported by worksheet #929

Open
rochala opened this issue Aug 21, 2023 · 0 comments
Open

Fewer braces are not fully supported by worksheet #929

rochala opened this issue Aug 21, 2023 · 0 comments

Comments

@rochala
Copy link
Collaborator

rochala commented Aug 21, 2023

Describe the bug

There are cases in which fewer braces fail in worksheet mode.

Scastie snippet link

No response

To Reproduce

List(1,2,3).map:
  case x => x

Expected behavior

It should properly instrument and show computed value: List(1,2,3)

This issue is caused by custom implementation of worksheet support which was written before fewer braces. As there is ongoing initiative to have shared implementation for worksheets for Scala, it is not worth duplicating the effort only to migrate in the near future.

The possible fix require changes in multiple part of application and new logic to take into account newly added lines leading to shifted error messages etc.

Screenshot ( Optional )

No response

Version

Scastie website

What browsers are you seeing the problem on?

No response

Relevant log output

No response

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant