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 340_lsp_support.md #26

Merged
merged 47 commits into from
Jul 8, 2024
Merged

Conversation

crapo
Copy link
Contributor

@crapo crapo commented Nov 30, 2023

Added brief instructions on how to add TextMate grammar.

Added brief instructions on how to add TextMate grammar.
@cdietrich
Copy link
Member

@crapo would be so kind an check if you still have a valid eca signed? https://www.eclipse.org/legal/ECA.php

@@ -72,7 +72,10 @@ Xtext supports the generation of language servers that comply to the [Language S
"uuid": "8383e49a-fa0d-4bb5-827b-10e8abb294ca"
}
```
1. Open a `*.dmodel` file with the Generic Editor and explore the supported language features like syntax highlighting, content assist, validation, displaying code lenses, quickfixes, formatting, ... :
You may save the content in a file and then add it to the TextMate grammar in Preferences.
![image](https://github.com/eclipse/xtext-website/assets/1186278/63b7eae9-6f9e-485c-bcb5-a052acce077c)
Copy link
Member

Choose a reason for hiding this comment

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

assume we need a relative path here

Copy link
Contributor

Choose a reason for hiding this comment

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

Let's see how it will be rendered on the webpage once it is merged.

Copy link
Member

Choose a reason for hiding this comment

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

wont help inside eclipse help system i assume

@crapo
Copy link
Contributor Author

crapo commented Dec 4, 2023

I've completed an ECA.

cdietrich and others added 16 commits January 18, 2024 05:58
Signed-off-by: Christian Dietrich <christian.dietrich.opensource@gmail.com>
Signed-off-by: Christian Dietrich <christian.dietrich.opensource@gmail.com>
Prepare Xtext/Xtend 2.35 release notes
Signed-off-by: miklossy <miklossy@itemis.de>
…_release-notes

Add "Testing" section to the Xtext 2.35.0 release notes.
Signed-off-by: miklossy <miklossy@itemis.de>
Co-authored-by: Sebastian Zarnekow <Sebastian.Zarnekow@gmail.com>
@LorenzoBettini
Copy link
Contributor

@cdietrich can we merge this one?

Normalization of EOLs in Xbase multi-line strings
@cdietrich
Copy link
Member

My question is not answered

@LorenzoBettini
Copy link
Contributor

which one?

@cdietrich
Copy link
Member

If it works in eclipse help
Don’t have a machine with me where I have the ruby stuff set up.

@LorenzoBettini
Copy link
Contributor

@crapo maybe it's better if you put the PNG image in the images directory like we did for other images

@miklossy
Copy link
Contributor

miklossy commented Jul 8, 2024

@crapo maybe it's better if you put the PNG image in the images directory like we did for other images

Since the PNG image has been put into the images directory (see 2101940), I think we could merge this PR.

@LorenzoBettini @cdietrich what do you think?

@cdietrich
Copy link
Member

i wonder if we can cleanup the commit history

@crapo
Copy link
Contributor Author

crapo commented Jul 8, 2024

If that is directed at me, I'm not sure exactly what that means or how to do it.

@cdietrich
Copy link
Member

yes i wonder if you can rebase and squash all your changes into one commit

@LorenzoBettini
Copy link
Contributor

@cdietrich we can do that ourselves when merging (Squash and merge)

@cdietrich
Copy link
Member

@LorenzoBettini if you could do that i am fine with it

@LorenzoBettini LorenzoBettini merged commit ea08d69 into eclipse:published Jul 8, 2024
1 check passed
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.

5 participants