Skip to content

Indentation Syntax #6

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
Draft
wants to merge 22 commits into
base: main
Choose a base branch
from
Draft

Indentation Syntax #6

wants to merge 22 commits into from

Conversation

kotleta2007
Copy link
Owner

This rule implements the new Scala 3 syntax for significant indentation defined here.

@kotleta2007 kotleta2007 requested a review from adpi2 October 30, 2023 10:25
@kotleta2007 kotleta2007 self-assigned this Oct 30, 2023
@kotleta2007
Implemented indentation syntax for control structures
35d0129
@kotleta2007
Implemented indentation syntax for case blocks (try-catch and pattern…
16483c8 
… matching)
@kotleta2007
Implemented new syntax for templates
07a182c
@kotleta2007
Added new tests
0e3e599
@kotleta2007
Added end markers and removing whitespace before colon
b9b2be0
@kotleta2007
Added RLEIndent and addEndMarker method
cb016cd
@kotleta2007
Added matchTreeTypeWithEndMarkers
e678fa3
@kotleta2007
Added splitOn; added end markers for if-statements
b6857b3
@kotleta2007
Added tests for end markers
dfbc57f
@kotleta2007
Added new end markers
d779ea3
@kotleta2007
Added support for modifier keywords
ced7ec7
@kotleta2007
Added support for extensions
f49954e
@kotleta2007
Added comparison operators for RLEIndent
5c1be18
@kotleta2007
Reference example of end markers works
3098c90
@kotleta2007
Commiting before refactoring indentation patches
f06a60f
@kotleta2007
Added brace patches
19f1ac3
@kotleta2007
replace right brace with end marker
6f82429
@kotleta2007
indentation works
ae64df9
@adpi2
Copy link
Collaborator

adpi2 commented Nov 29, 2023

https://github.com/lampepfl/dotty/pull/17522/files#diff-0d604955048586a7b83b98b4d4c0ba00c7b1a2c5ebd21a54f3a071ff4d40c13b

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@adpi2 adpi2 Awaiting requested review from adpi2

At least 1 approving review is required to merge this pull request.

Assignees

@kotleta2007 kotleta2007

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@kotleta2007 @adpi2