Skip to content

WIP adding CodeGenerator.group#984

Closed
RBusarow wants to merge 1 commit intomainfrom
rick/CodeGenerator_group
Closed

WIP adding CodeGenerator.group#984
RBusarow wants to merge 1 commit intomainfrom
rick/CodeGenerator_group

Conversation

@RBusarow
Copy link
Contributor

No description provided.

@RBusarow RBusarow force-pushed the rick/CodeGenerator_group branch 4 times, most recently from 39d357e to a720e2f Compare April 26, 2024 20:44
@RBusarow RBusarow marked this pull request as ready for review April 26, 2024 20:44
The `group` value allows for basic scheduling of code generators. All code generators with the same group will be executed in a loop together until no new code is generated.
@RBusarow RBusarow force-pushed the rick/CodeGenerator_group branch from a720e2f to b120e17 Compare July 9, 2024 19:21
* The default [group][CodeGenerator.group] for code generators.
* A lower group value will be executed before any higher group value.
*/
public const val GROUP_DEFAULT: Int = 0

Choose a reason for hiding this comment

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

Could use an enum for this, too! Integers for priority levels tend to lose information fairly readily. An enum would retain ordering guarantees, but enforce naming and make it a lot easier to nav through and find out where a particular CodeGenerator sits in the group.

@SteinerOk
Copy link

Hey @RBusarow, is it PR related with this my bug: #953 ?

@RBusarow RBusarow closed this Jun 3, 2025
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