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

Add additional tips & tricks to the sublime text documentation #906

Merged
merged 2 commits into from Sep 8, 2019

Conversation

@LukaszByczynski
Copy link
Contributor

commented Sep 7, 2019

As we agreed at Gitter channel I prepared some tips & tricks for better development productivity in Sublime Text.

Copy link
Member

left a comment

This looks great! Thank you @LukaszByczynski for writing up these instructions ❤️

cc/ @ayoub-benali could you maybe take a look?

@gabro
gabro approved these changes Sep 8, 2019
Copy link
Member

left a comment

🙏 amazing job, @LukaszByczynski, thank you so much!

I've left a few suggestions here and there, but big 👍 overall!

docs/editors/sublime.md Outdated Show resolved Hide resolved
docs/editors/sublime.md Outdated Show resolved Hide resolved
docs/editors/sublime.md Outdated Show resolved Hide resolved
docs/editors/sublime.md Outdated Show resolved Hide resolved
docs/editors/sublime.md Outdated Show resolved Hide resolved
docs/editors/sublime.md Outdated Show resolved Hide resolved
docs/editors/sublime.md Outdated Show resolved Hide resolved
@LukaszByczynski

This comment has been minimized.

Copy link
Contributor Author

commented Sep 8, 2019

Thanks @gabro for suggestions and all of you for a fast review.

@gabro gabro merged commit 9c56940 into scalameta:master Sep 8, 2019
2 checks passed
2 checks passed
continuous-integration/travis-ci/pr The Travis CI build passed
Details
scalameta.metals Build #20190908.1 succeeded
Details
]
```

### Enabling auto-import on completion

This comment has been minimized.

Copy link
@ayoub-benali

ayoub-benali Sep 9, 2019

Contributor

I believe this isn't necessary since LSP implements additionalTextEdits via tomv564/LSP#602
Since several months auto import works for me without needing the mapping bellow. Or am I missing something new ?

This comment has been minimized.

Copy link
@gabro

gabro Sep 9, 2019

Member

Woops, I've merged without waiting for your review @ayoub-benali, sorry 😓

@LukaszByczynski can you double check this?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
4 participants
You can’t perform that action at this time.