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.

Sign up for GitHub

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

Jump to bottom

In the `GlueType` AST we should probably have a `UtilityType` case to regroup the different utility together #110

Closed

MangelMaxime opened this issue Jul 15, 2024 · 0 comments

Labels

Contributor

MangelMaxime commented Jul 15, 2024

The reason behind this idea is to avoid polluting too much the main AST branch.

[<RequireQualifiedAccess>]
type GlueType =
	| Discard
    | Interface of GlueInterface
	// .. 
	| Record of GlueType * GlueType
	| Partial of GlueType
	| ReadOnly of GlueType
	// ..

would becomes

[<RequireQualifiedAccess>]
type UtilityType =
	| Record of GlueType * GlueType
	| Partial of GlueType
	| ReadOnly of GlueType

[<RequireQualifiedAccess>]
type GlueType =
	| Discard
    | Interface of GlueInterface
	| UtilityType of UtilityType
	// ..

MangelMaxime added the area:converter label

MangelMaxime closed this as completed in

70a7ea9

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment