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

Implement FunctionInfo (1 of 3) #323

Merged
merged 5 commits into from
Jul 20, 2022
Merged

Implement FunctionInfo (1 of 3) #323

merged 5 commits into from
Jul 20, 2022

Conversation

unode
Copy link
Collaborator

@unode unode commented Jul 15, 2022

This is an attempt to split #226 into smaller PRs, easier to review.
This PR should be merged before the others.

However due to how dependent these are from each other, it's not easy to ensure each PR is self consistent.
The glue code will be part of the original #226 PR.

Allows passing custom function annotations that can be used
by the help plugin to format or group functions

(cherry picked from commit 66bb750)
(cherry picked from commit 8500e031ab34de5ead988e68fb41d4de6a4968a0)
Defines an interface for listeners to allow easier collection and formatting of help information
@codeclimate
Copy link

codeclimate bot commented Jul 15, 2022

Code Climate has analyzed commit b51a52a and detected 0 issues on this pull request.

View more on Code Climate.

@attzonko attzonko merged commit 73f081f into attzonko:main Jul 20, 2022
unode added a commit to unode/mmpy_bot that referenced this pull request Feb 9, 2023
* Add wildcard metadata attribute to Function

Allows passing custom function annotations that can be used
by the help plugin to format or group functions

* Document extra metadata for help usage in listeners

* Add is_click_function to Function for API consistency

* Add FunctionInfo object

Defines an interface for listeners to allow easier collection and formatting of help information

* Document FunctionInfo objects and how to customize help messages
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