Filepath.abspath are directory' path resolving fail fix

[tsuyoshicho]

環境 Windows
ターゲットpath D:\data
pwd D:\data\test

の状態で
FIlepath.abspathに、\Data\を入れると D:\Data\test\Data\のようになる
(カレントドライブレターはfinddir他で消えることがあるので...fnamemodifyで直ることを期待した)

これは、\Data\がisdirecotryではあってもfilereadableではないためのようです。
逆に上記の状態で\Data\はisdirectoryで1が返ります。そして手動でfnamemodifyするとフルパスが取れます。

filereadable or isdirectoryが条件でもいいのではないか、と思うのですが、どうでしょうか?
ドキュメントにも

abspath({path}) Vital.System.Filepath.abspath()
Return an absolute path of {path}.
If the {path} is already an absolute path, it return the {path}.
cf. |Vital.System.Filepath.relpath()|

とあり、pathであってfile、という制限ではないように読めました。

関数

function! s:abspath(path) abort
  if s:is_absolute(a:path)
    return a:path
  endif
  " Note:
  "   the behavior of ':p' for non existing file path is not defined
  return filereadable(a:path)
        \ ? fnamemodify(a:path, ':p')
        \ : s:join(fnamemodify(getcwd(), ':p'), a:path)
endfunction

テスト用

:echo isdirectory('\Data\')
:echo filereadable('\Data\')

[ujihisa]

Assigned the reviewers to the maintainers

[tyru]

@tsuyoshicho maser を merge するか rebase してもらえますか？

[tsuyoshicho]

了解です。

[tsuyoshicho]

やっぱり、なんかreviewdogのtokenでだめですね...

[tyru]

以前 #610 で token の再生成を行った記憶があるんですが、やり方を失念してしまった…
#610 でもとりあえず訳も分からず再生成したので、原因とか分かりますかね？ (cc: @haya14busa)

http://droneio.haya14busa.com/ からログインしようとしたら

Unable to login. Registration is closed.

って出てログインできませんでした。

[tyru]

github が不調だったのでそのせいかもですが、復帰した今も同じ症状です。

[tyru]

なんかよくわからんけど push したら直ったようで良かったです（こなみ）
とりあえず issue だけ立てておきました。 #684
この PR はマージしちゃいます。

[haya14busa]

droneio はもう使ってないので全く関係ないです。 401 Bad credentials がでるのは謎です。
正確に言うとreviewdogのトークンではなく後ろで取得したGitHub API token がなぜかGitHub APIから401にランダムに弾かれてるのでreviewdogトークンを再生成しても意味はありません。

[tyru]

なるほど… つらそう…
とするとどうすればいいんでしょう…
ひたすら空の commit を push して re-run させる？

[tsuyoshicho]

GitHub APIのアクセス制限の類なのでしょうか...?

[tyru]

@vim-jp/vim-vimlparser
https://github.com/vim-jp/vital.vim/settings/hooks から drone.io のやつ消しちゃいますね (土日とかに)

[mattn]

OK です。

[tyru]

消しました（忘れてた）