[Feature] Improve `Path:normalize` to return shorter relative path using ../
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 4/5
- Tiempo estimado
- 3-5 días
- Aptitud para principiantes
- 35/100
Línea de trabajo
Locate plenary.path's Path:normalize implementation and any related path tests. Compare its existing absolute and relative-down results with the proposed relative-up form, while accounting for Windows support. Done means normalize returns the shortest valid path without breaking existing behavior.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Currently Path:normalize returns 3 possible paths:
- absolute from /
- absolute from ~
- relative down from cwd
I propose to add a fourth one:
- relative up from cwd
At a glance this changes behaviour in a compatible way and shouldn't break any uses.
The point is to get the shortest possible path (which is anyway feels like a current goal for this method) - sometimes it may be the absolute one, sometimes up from cwd.
I've found it really useful in Neovim, when file is opened using Telescope plugin and you doesn't control how file name will be formed (manually you can either do :tabnew ../file or :tabnew ~/proj/file), but you wanna see as short as possible file name in statusline/tab name. Here is my current implementation (as a Lazy plugin):
---@type LazySpec
return {
{
'nvim-lua/plenary.nvim', -- Not a plugin, just a useful library.
config = function()
local Path = require 'plenary.path'
local normalize = Path.normalize
--- Monkey-patch Path:normalize method to make it try harder looking for shortest
--- relative path by checking also path UP from cwd: '../../…'.
---@diagnostic disable-next-line: duplicate-set-field
Path.normalize = function(self, cwd)
-- Absolute (DOWN FROM / or ~) or relative (DOWN FROM cwd).
local orig = normalize(self, cwd)
-- Absolute (DOWN FROM / or ~), but we'll make it relative (UP FROM cwd).
local rel = vim.fn.fnamemodify(orig, ':p:~')
if string.match(orig, '^[/~]') then -- Absolute, thus may be shorter.
local abs = vim.fn.fnamemodify(rel, ':p')
local abs_path = Path:new(abs)
local dir = cwd .. '/'
local up = ''
repeat
up = up .. '../'
rel = abs_path:make_relative(Path:new(dir .. up):absolute())
until rel ~= abs
rel = up .. rel
end
return string.len(orig) <= string.len(rel) and orig or rel
end
end,
},
}
This implementation is not suitable for the lib (because there is no Windows support and it depends on Neovim), but sometimes code worth thousands words 😄 and also it may be useful for other Neovim users until this change will be implemented (if it will be accepted at all).
- Lenguaje dominante
- Lua
- Estrellas
- 3.5k
- Forks
- 341
- Métricas de merge de PR
- Sin PR fusionados en 30 d
Preparar el entorno
Este proyecto no incluye contenedor de desarrollo, Dockerfile ni guía de contribución, así que la configuración corre por tu cuenta: empieza por su README y consulta nuestra guía para la primera contribución para los pasos generales.
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 nvim-lua/plenary.nvim
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 88/100
nvim-lua/plenary.nvim#682 ·
-
uniform usage of `os.getenv`Posiblemente ocupada @rakotomandimby la tomó hace 124 días. Abierto
Dificultad 1/5 Menos de una hora Aptitud para principiantes 72/100
nvim-lua/plenary.nvim#680 ·
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 45/100
nvim-lua/plenary.nvim#675 ·
-
Dificultad 4/5 3-5 días Aptitud para principiantes 28/100
nvim-lua/plenary.nvim#672 · 1 comentario ·
-
Dificultad 4/5 3-5 días Aptitud para principiantes 25/100
nvim-lua/plenary.nvim#671 · 1 comentario ·
Todos los issues de nvim-lua/plenary.nvim
Issues similares
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 70/100
-
bug
Dificultad 1/5 1-3 horas Aptitud para principiantes 85/100
bradcypert/plum#265 ·
Los mantenedores suelen responder en 1 día
-
Lasso not PastingAbiertobug
Dificultad 2/5 1-3 horas Aptitud para principiantes 65/100
-
code:todo
Dificultad 1/5 Menos de una hora Aptitud para principiantes 72/100
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 83/100
koreader/koreader#16221 · 1 comentario ·
Los mantenedores suelen responder en 1 día