-
Notifications
You must be signed in to change notification settings - Fork 4
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 Lionweb and the Starlasu implementation in Lionweb #83
base: master
Are you sure you want to change the base?
Conversation
…st Kolasu 1.5, also addressing #79
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I wonder if we should have a JSON file saved somewhere and just load it here (just sharing a thought, I do not expect this to be changed in this PR)
@@ -29,7 +30,7 @@ import {STARLASU_LANGUAGE} from "./lionweb-starlasu-language"; | |||
export {STARLASU_LANGUAGE} from "./lionweb-starlasu-language"; | |||
|
|||
export const ASTNode = STARLASU_LANGUAGE.entities.find(e => e.name == "ASTNode")! as Concept; | |||
export const PositionFeature = ASTNode.features.find(f => f.name == "position")! as Containment; | |||
export const PositionFeature = ASTNode.features.find(f => f.name == "position")! as Property; | |||
export const PositionClassifier = PositionFeature.type!; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I would not call those Classifiers as Classifier is a Concept, an Interface or an Annotation. This is a PrimitiveType
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
All seems in order
Addresses #79