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

Add maestro check-syntax command #2387

Merged
merged 1 commit into from
Mar 20, 2025
Merged

Conversation

Leland-Takamine
Copy link
Contributor

@Leland-Takamine Leland-Takamine commented Mar 20, 2025

✔️ Single command

$ echo "tapOn: foo" | maestro check-syntax -

OK

✔️ Full flow

$ cat << EOF  | maestro check-syntax -
appId: com.example.app
---
- tapOn: foo
EOF

OK

✔️ List of commands

$ cat << EOF  | maestro check-syntax -
- tapOn: foo
- inputText: bar
EOF

OK

✔️ File input

$ maestro check-syntax MyFlow.yaml

OK

✔️ Errors

$ cat << EOF  | maestro check-syntax -
- tapOn: foo
- inp
EOF

/syntax-checker:2
╭────────────────────────────────────────────────────────────────────────────────╮
│ 1 | - tapOn: foo                                                               │
│ 2 | - inp                                                                      │
│          ^                                                                     │
│ ╭────────────────────────────────────────────────────────────────────────────╮ │
│ │ `inp` is not a valid command.                                              │ │
│ │                                                                            │ │
│ │ Did you mean one of: inputRandomText, inputRandomNumber, inputRandomEmail, │ │
│ │ inputRandomPersonName, inputText                                           │ │
│ │                                                                            │ │
│ │ > https://docs.maestro.dev/api-reference/commands                          │ │
│ ╰────────────────────────────────────────────────────────────────────────────╯ │
│ 3 |                                                                            │
╰────────────────────────────────────────────────────────────────────────────────╯

@Leland-Takamine Leland-Takamine requested a review from herval March 20, 2025 21:18
@Leland-Takamine Leland-Takamine merged commit 18809cc into main Mar 20, 2025
8 checks passed
@Leland-Takamine Leland-Takamine deleted the check-syntax-command branch March 20, 2025 21:36
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.

1 participant