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

Feature request: Extend M122 Pnnn to generate tests for all event types #895

Closed
T3P3 opened this issue Aug 21, 2023 · 3 comments
Closed
Assignees

Comments

@T3P3
Copy link
Contributor

T3P3 commented Aug 21, 2023

In order to test event handling macros it would be useful to have a P parameter, similar to M122 P109 that generates each of the event types. This could either be, for example P110 for filament-error P111 for driver-warning etc or MP110 S"driver-warning" etc Ideally the D, P,C parameters, if applicable, could be passed as well to specify the source and type of sub-event

@dc42
Copy link
Collaborator

dc42 commented Sep 7, 2023

@T3P3
Copy link
Contributor Author

T3P3 commented Sep 7, 2023

Yes it is, I will make it more obvious in the event documentation that this feature exists.

@T3P3 T3P3 closed this as completed Sep 7, 2023
@T3P3
Copy link
Contributor Author

T3P3 commented Sep 11, 2023

documentation updated

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

No branches or pull requests

2 participants