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

Update the state machine specification for do actions as lists #482

Merged

Conversation

garthwatney
Copy link
Collaborator

@garthwatney garthwatney commented Jul 25, 2024

  • Factor the do syntax into a separate "do expression"
  • Replace optional action with action list in do expressions
  • Add entry and exit actions

@garthwatney garthwatney requested a review from bocchino July 25, 2024 18:39
Copy link
Collaborator

@bocchino bocchino left a comment

Choose a reason for hiding this comment

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

Looks good! I added some comments. We need to add the new keywords in the lexical spec and in the other places (editors, code prettify).

@garthwatney garthwatney requested a review from bocchino July 30, 2024 20:51
Copy link
Collaborator

@bocchino bocchino left a comment

Choose a reason for hiding this comment

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

Looks good!

@bocchino bocchino merged commit 7637728 into nasa:feature/state-machine-phase-2 Jul 31, 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.

2 participants