refactor: `Plugin.uri` => `Plugin.url`

This commit is contained in:
Folke Lemaitre 2022-12-06 11:12:54 +01:00
parent 43b303bd8f
commit 4cfe0b5315
No known key found for this signature in database
GPG Key ID: 41F8B1FBACAE2040
8 changed files with 27 additions and 27 deletions

View File

@ -36,7 +36,7 @@ local M = {}
---@class LazyPlugin: LazyPluginHandlers,LazyPluginHooks,LazyPluginRef ---@class LazyPlugin: LazyPluginHandlers,LazyPluginHooks,LazyPluginRef
---@field [1] string ---@field [1] string
---@field name string display name and name used for plugin config files ---@field name string display name and name used for plugin config files
---@field uri string ---@field url string
---@field dir string ---@field dir string
---@field enabled? boolean|(fun():boolean) ---@field enabled? boolean|(fun():boolean)
---@field lazy? boolean ---@field lazy? boolean
@ -69,14 +69,14 @@ function Spec:add(plugin, is_dep)
Util.error("Invalid plugin spec " .. vim.inspect(plugin)) Util.error("Invalid plugin spec " .. vim.inspect(plugin))
end end
if not plugin.uri then if not plugin.url then
local c = pkg:sub(1, 1) local c = pkg:sub(1, 1)
if c == "~" then if c == "~" then
plugin.uri = vim.loop.os_getenv("HOME") .. pkg:sub(2) plugin.url = vim.loop.os_getenv("HOME") .. pkg:sub(2)
elseif c == "/" or pkg:sub(1, 4) == "http" or pkg:sub(1, 3) == "ssh" then elseif c == "/" or pkg:sub(1, 4) == "http" or pkg:sub(1, 3) == "ssh" then
plugin.uri = pkg plugin.url = pkg
else else
plugin.uri = ("https://github.com/" .. pkg .. ".git") plugin.url = ("https://github.com/" .. pkg .. ".git")
end end
end end
@ -93,7 +93,7 @@ function Spec:add(plugin, is_dep)
-- check for plugins that should be local -- check for plugins that should be local
for _, pattern in ipairs(Config.options.dev.patterns) do for _, pattern in ipairs(Config.options.dev.patterns) do
if plugin.dev or (plugin[1]:find(pattern, 1, true) and plugin.dev ~= false) then if plugin.dev or (plugin[1]:find(pattern, 1, true) and plugin.dev ~= false) then
plugin.uri = Config.options.dev.path .. "/" .. plugin.name plugin.url = Config.options.dev.path .. "/" .. plugin.name
break break
end end
end end
@ -171,9 +171,9 @@ function M.update_state()
or plugin.cmd or plugin.cmd
plugin.lazy = lazy and true or false plugin.lazy = lazy and true or false
end end
if plugin.uri:sub(1, 4) ~= "http" and plugin.uri:sub(1, 3) ~= "git" then if plugin.url:sub(1, 4) ~= "http" and plugin.url:sub(1, 3) ~= "git" then
plugin._.is_local = true plugin._.is_local = true
plugin.dir = plugin.uri plugin.dir = plugin.url
plugin._.installed = true -- user should make sure the directory exists plugin._.installed = true -- user should make sure the directory exists
else else
plugin.dir = Config.options.root .. "/" .. plugin.name plugin.dir = Config.options.root .. "/" .. plugin.name

View File

@ -25,7 +25,7 @@ function M.check()
local valid = { local valid = {
1, 1,
"name", "name",
"uri", "url",
"enabled", "enabled",
"lazy", "lazy",
"dev", "dev",

View File

@ -62,7 +62,7 @@ function M.install(opts)
"plugin.build", "plugin.build",
}, },
plugins = function(plugin) plugins = function(plugin)
return plugin.uri and not plugin._.installed return plugin.url and not plugin._.installed
end, end,
}, opts) }, opts)
end end
@ -81,7 +81,7 @@ function M.update(opts)
{ "git.log", updated = true }, { "git.log", updated = true },
}, },
plugins = function(plugin) plugins = function(plugin)
return plugin.uri and plugin._.installed return plugin.url and plugin._.installed
end, end,
}, opts):wait(function() }, opts):wait(function()
require("lazy.manage.lock").update() require("lazy.manage.lock").update()
@ -98,7 +98,7 @@ function M.check(opts)
{ "git.log", check = true }, { "git.log", check = true },
}, },
plugins = function(plugin) plugins = function(plugin)
return plugin.uri and plugin._.installed return plugin.url and plugin._.installed
end, end,
}, opts) }, opts)
end end
@ -108,7 +108,7 @@ function M.log(opts)
return M.run({ return M.run({
pipeline = { "git.log" }, pipeline = { "git.log" },
plugins = function(plugin) plugins = function(plugin)
return plugin.uri and plugin._.installed return plugin.url and plugin._.installed
end, end,
}, opts) }, opts)
end end

View File

@ -51,7 +51,7 @@ M.clone = {
run = function(self) run = function(self)
local args = { local args = {
"clone", "clone",
self.plugin.uri, self.plugin.url,
"--filter=blob:none", "--filter=blob:none",
"--recurse-submodules", "--recurse-submodules",
"--single-branch", "--single-branch",

View File

@ -130,7 +130,7 @@ function M.show(mode)
local function open(path) local function open(path)
local plugin = get_plugin() local plugin = get_plugin()
if plugin then if plugin then
local url = plugin.uri:gsub("%.git$", "") local url = plugin.url:gsub("%.git$", "")
if Util.file_exists(url) then if Util.file_exists(url) then
url = "https://github.com/" .. plugin[1] url = "https://github.com/" .. plugin[1]
end end

View File

@ -363,7 +363,7 @@ end
function M:details(plugin) function M:details(plugin)
---@type string[][] ---@type string[][]
local props = {} local props = {}
table.insert(props, { "uri", (plugin.uri:gsub("%.git$", "")), "@text.reference" }) table.insert(props, { "url", (plugin.url:gsub("%.git$", "")), "@text.reference" })
local git = Git.info(plugin.dir, true) local git = Git.info(plugin.dir, true)
if git then if git then
git.branch = git.branch or Git.get_branch(plugin) git.branch = git.branch or Git.get_branch(plugin)

View File

@ -70,7 +70,7 @@ return {
}, },
{ {
filter = function(plugin) filter = function(plugin)
return plugin._.installed and not plugin.uri return plugin._.installed and not plugin.url
end, end,
title = "Clean", title = "Clean",
}, },

View File

@ -5,17 +5,17 @@ local assert = require("luassert")
Config.setup() Config.setup()
describe("plugin spec uri/name", function() describe("plugin spec url/name", function()
local tests = { local tests = {
{ { "~/foo" }, { [1] = "~/foo", name = "foo", uri = vim.fn.fnamemodify("~/foo", ":p") } }, { { "~/foo" }, { [1] = "~/foo", name = "foo", url = vim.fn.fnamemodify("~/foo", ":p") } },
{ { "/tmp/foo" }, { [1] = "/tmp/foo", name = "foo", uri = "/tmp/foo" } }, { { "/tmp/foo" }, { [1] = "/tmp/foo", name = "foo", url = "/tmp/foo" } },
{ { "foo/bar" }, { [1] = "foo/bar", name = "bar", uri = "https://github.com/foo/bar.git" } }, { { "foo/bar" }, { [1] = "foo/bar", name = "bar", url = "https://github.com/foo/bar.git" } },
{ { "foo/bar", name = "foobar" }, { [1] = "foo/bar", name = "foobar", uri = "https://github.com/foo/bar.git" } }, { { "foo/bar", name = "foobar" }, { [1] = "foo/bar", name = "foobar", url = "https://github.com/foo/bar.git" } },
{ { "foo/bar", uri = "123" }, { [1] = "foo/bar", name = "bar", uri = "123" } }, { { "foo/bar", url = "123" }, { [1] = "foo/bar", name = "bar", url = "123" } },
{ { "https://foobar" }, { [1] = "https://foobar", name = "foobar", uri = "https://foobar" } }, { { "https://foobar" }, { [1] = "https://foobar", name = "foobar", url = "https://foobar" } },
{ { "ssh://foobar" }, { [1] = "ssh://foobar", name = "foobar", uri = "ssh://foobar" } }, { { "ssh://foobar" }, { [1] = "ssh://foobar", name = "foobar", url = "ssh://foobar" } },
{ "foo/bar", { [1] = "foo/bar", name = "bar", uri = "https://github.com/foo/bar.git" } }, { "foo/bar", { [1] = "foo/bar", name = "bar", url = "https://github.com/foo/bar.git" } },
{ { { { "foo/bar" } } }, { [1] = "foo/bar", name = "bar", uri = "https://github.com/foo/bar.git" } }, { { { { "foo/bar" } } }, { [1] = "foo/bar", name = "bar", url = "https://github.com/foo/bar.git" } },
} }
for _, test in ipairs(tests) do for _, test in ipairs(tests) do