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

MAINT: refactor in cleanup in view of #13879 (shortcut configuration). #13894

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

Carreau
Copy link
Member

@Carreau Carreau commented Jan 16, 2023

We will likely need a data-mapping shortcut to filter and commands, this commands (functions), need to not be closures for delayed binding, and we want non-complex filter I'd like to avoid in the configuration having syntax like (foo | bar) & baz.

…ration).

We will likely need a data-mapping shortcut to filter and commands,
this commands (functions), need to not be closures for delayed binding,
and we want non-complex filter I'd like to avoid in the configuration
having syntax like `(foo | bar) & baz`.
@krassowski
Copy link
Member

this commands (functions), need to not be closures for delayed binding

did you see my proposed changes in #13888 which I think go in that direction?

@Carreau
Copy link
Member Author

Carreau commented Jan 16, 2023 via email

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.

None yet

2 participants