Pass the workspace_builder to plugin hooks instead of the tmux session
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 5/5
- Tiempo estimado
- Más de una semana
- Aptitud para principiantes
- 35/100
Línea de trabajo
No files or tests are named. Start by locating the plugin hooks and comparing their current session argument with the workspace_builder interface; clarify the compatibility approach with maintainers. Done means hooks can access workspace configuration through workspace_builder without leaving the chosen legacy-hook behavior ambiguous.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
First of all, thank you for this app that changed my life with tmux !
In the current plugin system, it seems impossible to access the session configuration (from yaml or json).
Is there a specific reason the workspace is not passed in the hooks ?
...This would allow to easily extend the configuration file format.
I suggest to replace the session argument with a workspace_builder argument (which already gives access to the session) in the hooks. The drawback is that it's a breaking change. Here are some ways to overcome it:
- Duplicate the hooks
- Have the new hook call the legacy one by default
- Call both the hooks
- Add a new plugin class with hooks expecting the workspace builder
I can of course make a PR myself, but I need to know which way you prefer
- Lenguaje dominante
- Python
- Estrellas
- 4.6k
- Forks
- 247
- Métricas de merge de PR
- Sin PR fusionados en 30 d
Preparar el entorno
- Sin Dockerfile ni archivo de Docker Compose
- Sin plantilla de pull request
- Leer la guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de tmux-python/tmuxp
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
tmux-python/tmuxp#1026 · 1 comentario ·
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 65/100
tmux-python/tmuxp#939 · 1 comentario ·
-
enhancement WorkspaceBuilder
Dificultad 4/5 3-5 días Aptitud para principiantes 48/100
tmux-python/tmuxp#1062 ·
-
enhancement need-comments WorkspaceBuilder
Dificultad 5/5 Más de una semana Aptitud para principiantes 35/100
tmux-python/tmuxp#1058 · 2 comentarios ·
-
Very slow performance opening layouts/workspacesPosiblemente ocupada Un pull request vinculado a esta issue está abierto o ya se fusionó. Abiertobug
Dificultad 4/5 3-5 días Aptitud para principiantes 45/100
tmux-python/tmuxp#1053 · 10 comentarios ·
Todos los issues de tmux-python/tmuxp
Issues similares
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 85/100
Los mantenedores suelen responder en 3 días
-
Negation with "not" and "no" is ignored during sentiment analysisPosiblemente ocupada @vivek-3728 la tomó hoy. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
techcsispit/mess-mood#11 · 1 comentario ·
-
changelog investigate
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
ramnes/notion-sdk-py#408 ·
-
good first issue
Dificultad 2/5 1-3 horas Aptitud para principiantes 83/100
btclib-org/btclib-wallet#267 ·
Los mantenedores suelen responder en 1 día
-
good first issue tech-debt
Dificultad 2/5 1-3 horas Aptitud para principiantes 85/100
knnmelprop/YAADO#111 ·