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

Use correct module type for generated module #1393

Merged
merged 2 commits into from
Feb 29, 2024

Conversation

msujew
Copy link
Member

@msujew msujew commented Feb 29, 2024

Related to #1392

Instead of being part of LangiumServices, the module should instead be part of LangiumCoreServices. Afaik, the current solution is working fine, but using LangiumCoreServices is more appropriate.

@msujew msujew requested a review from sailingKieler February 29, 2024 15:31
Copy link
Contributor

@sailingKieler sailingKieler left a comment

Choose a reason for hiding this comment

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

Indeed, we missed that one. 🧐
One could change that manually, but regenerating the generated code breaks it, in case the language impl has been switched to 'core' only.

The actual change is just in the template in langium-cli, isn't it?

@msujew
Copy link
Member Author

msujew commented Feb 29, 2024

The actual change is just in the template in langium-cli, isn't it?

Yes, the rest is just regenerated code 👍

@msujew msujew merged commit 18a38ce into main Feb 29, 2024
5 checks passed
@spoenemann spoenemann added this to the v3.1.0 milestone Jun 17, 2024
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.

3 participants