Skip to content

Follow Scala 2.13's lead and make postfixOps syntax an error (not just a warning) unless the feature is explicitly enabled #5498

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
Closed
smarter opened this issue Nov 22, 2018 · 3 comments · Fixed by #8388
Closed

Follow Scala 2.13's lead and make postfixOps syntax an error (not just a warning) unless the feature is explicitly enabled #5498

smarter opened this issue Nov 22, 2018 · 3 comments · Fixed by #8388
Labels
area:reporting Error reporting including formatting, implicit suggestions, etc compat:scala2 good first issue Perfect for someone who wants to get started contributing help wanted

Comments

@smarter
Copy link
Member

smarter commented Nov 22, 2018

See scala/scala#6831
@smarter smarter added help wanted compat:scala2 area:reporting Error reporting including formatting, implicit suggestions, etc labels Nov 22, 2018
@smarter
Copy link
Member Author

smarter commented Nov 22, 2018

Also, assuming support for ancient sbt is not required, per scala/scala#7319

No, we only support sbt 1, and the sbt bridge is in this repository anyway (https://github.com/lampepfl/dotty/tree/master/sbt-bridge)

@anatoliykmetyuk anatoliykmetyuk added the good first issue Perfect for someone who wants to get started contributing label Feb 25, 2020
@robstoll robstoll mentioned this issue Feb 26, 2020
Merged
@SethTisue
Copy link
Member

or perhaps just don't support it at all?

@som-snytt som-snytt mentioned this issue Feb 26, 2020
Merged
@robstoll
Copy link
Contributor

I work on this but need some input -- i.a. if I shall enable the feature for dotty itself or rewrite code. See #8388 for the other questions

smarter added a commit that referenced this issue Mar 28, 2020
@smarter
Merge pull request #8388 from tegonal/#5498-postFix-error
3d17b02 
Fix #5498: error for postfixOps if language feature not enabled
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
area:reporting Error reporting including formatting, implicit suggestions, etc compat:scala2 good first issue Perfect for someone who wants to get started contributing help wanted
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fix #5498: error for postfixOps if language feature not enabled tegonal/dotty
4 participants
@smarter @SethTisue @anatoliykmetyuk @robstoll