Skip to content
This repository has been archived by the owner on Jan 1, 2023. It is now read-only.

Fixed incorrect menu cmd reprs #61

Merged
merged 1 commit into from
Sep 24, 2022
Merged

Fixed incorrect menu cmd reprs #61

merged 1 commit into from
Sep 24, 2022

Conversation

fdnt7
Copy link
Owner

@fdnt7 fdnt7 commented Sep 24, 2022

* get_pref(...) -> get_implied_prefix(...)
| * now also accepts cmd objs
| + now support returning '/' and '|>' for slash and menu cmds respectively
- removed deprecated overloads from get_full_cmd_repr_from_identifier(...)
? some minor code optimizations & refractoring
? _FlattenedT -> _RecursedT, FlattenerSig -> RecurserSig
? renamed some vars with more meaningful names

`*` `get_pref(...)` -> `get_implied_prefix(...)`
 | `*` now also accepts cmd objs
 | `+` now support returning `'/'` and `'|>'` for slash and menu cmds respectively
`-` removed deprecated overloads from `get_full_cmd_repr_from_identifier(...)`
`?` some minor code optimizations & refractoring
`?` `_FlattenedT` -> `_RecursedT`, `FlattenerSig` -> `RecurserSig`
`?` renamed some vars with more meaningful names
@fdnt7 fdnt7 added bug Something isn't working documentation Improvements or additions to documentation ux This change will affect the user experience labels Sep 24, 2022
@fdnt7 fdnt7 self-assigned this Sep 24, 2022
@fdnt7 fdnt7 merged commit 44138f4 into main Sep 24, 2022
@fdnt7 fdnt7 deleted the improve-menu-cmd-reprs branch September 24, 2022 13:16
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Something isn't working documentation Improvements or additions to documentation ux This change will affect the user experience
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant