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 to Monaco Editor version 0.13.1 #98

Merged
merged 8 commits into from
Sep 24, 2018
Merged

Conversation

jonas
Copy link
Collaborator

@jonas jonas commented Sep 24, 2018

Also cleans up several places to make it easier to update to a newer version.

I didn't manage to update to the latest version (0.14.1). The go to definition provider is not able to switch to another file.

@jonas jonas requested a review from olafurpg September 24, 2018 00:29
)
val symbol = jsObject[SymbolInformation]
symbol.name = denotation.displayName
// TODO: pretty print `.tpe`: https://github.com/scalameta/scalameta/issues/1479
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The issue has since been closed. Not sure if it is relevant anymore.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I've updated the comment to link to #99

resolveLocation no longer needs a model to do its work which allows
to simplify for-comprehensions in the providers calling it.
Add notes to the modifications of the bindings.
Copy link
Member

@olafurpg olafurpg left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice! That was a lot of work. Just tried locally and it works great 👍

Are there exciting new features that are unlocked by this upgrade? I can't find release notes for the Monaco editor.

I didn't manage to update to the latest version (0.14.1). The go to definition provider is not able to switch to another file.

That's a shame. Have the editor service internals changed? Can you maybe open a ticket if you have a branch I can try out? I wish Monaco had out of the box support for multi-file workspaces...

)
val symbol = jsObject[SymbolInformation]
symbol.name = denotation.displayName
// TODO: pretty print `.tpe`: https://github.com/scalameta/scalameta/issues/1479
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I've updated the comment to link to #99

@olafurpg
Copy link
Member

Restarted the CI job for 0.13 due to a timeout.

In case 0.13 continues to cause problems, I'm happy to drop it and only support sbt 1.0.

@olafurpg olafurpg merged commit 122f657 into master Sep 24, 2018
@olafurpg olafurpg deleted the monaco-editor-0.13.1 branch September 24, 2018 08:15
@jonas
Copy link
Collaborator Author

jonas commented Sep 24, 2018

I can make a branch with the changes to upgrade to the latest version.

The release notes are here https://github.com/Microsoft/monaco-editor/blob/master/CHANGELOG.md

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

Successfully merging this pull request may close these issues.

2 participants