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

Update Sublime settings since it does have auto-indent #1064

Merged
merged 1 commit into from Nov 12, 2019

Conversation

@ckipp01
Copy link
Member

ckipp01 commented Nov 12, 2019

I was doing some testing with this pr sublimelsp/LSP#772 to see how it would work with the new "implement all memebers" and it seems that Submlime does put in the auto-ident just like VS-Code and Atom does. I originally had this marked to false for Sublime causing it to look like so in Sublime

object Main {
	trait Foo {
		def foo: Int
		def bar: Int
		def car: Int
	}

	val x = new Foo {
		def foo: Int = ???
		  def bar: Int = ???
		  def car: Int = ???
	}
}

This will fix that.

@ckipp01 ckipp01 changed the title update submlime settings since it does have auto-indent Update submlime settings since it does have auto-indent Nov 12, 2019
@gabro
gabro approved these changes Nov 12, 2019
Copy link
Member

gabro left a comment

LGTM 👍

@ckipp01 ckipp01 changed the title Update submlime settings since it does have auto-indent Update Sublime settings since it does have auto-indent Nov 12, 2019
Copy link
Member

olafurpg left a comment

LGTM 👍 Thank you for following up on this!

@olafurpg olafurpg merged commit 05017dc into scalameta:master Nov 12, 2019
9 checks passed
9 checks passed
Windows unit tests
Details
Linux unit tests
Details
Sbt integration
Details
Maven integration
Details
Gradle integration
Details
Mill integration
Details
Slow tests
Details
Scala cross tests
Details
Scalafmt/Scalacheck/Docs
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants
You can’t perform that action at this time.