finos / rune-dsl

The project containing the Rune DSL grammar and default code generators
Apache License 2.0
24 stars 29 forks source link

Xtext grammar file is too large #756

Open SimonCockx opened 1 month ago

SimonCockx commented 1 month ago

Currently the whole grammar of Rune is defined in a single huge xtext file. This in turn leads to large derived Java classes, both generated (parser, sequencer, ...) and manually written (validator, formatter, ...).

Splitting up our grammar file can improve modularity, leading to more maintainable code. Additionally, projects using jacoco will stop seeing MethodTooLargeExceptions while building when they have the Rune DSL as a dependency.

I would propose the following splitting:

cdietrich commented 1 month ago

You should have a look in the workflow options to split classes eg the parser