v 0.1.23 removed filepath module

xucs007 commented 4 years ago

Now has os.ext

taojy123 commented 4 years ago

谢谢

taojy123 commented 4 years ago

不好意思，早上没仔细看你的 PR，其实不是 0.1.23 移除了 filepath，而是最新的 0.1.24 增加了 filepath.ext 来替代 os.ext，而 os.ext 即将被废弃。相关的更新之前在这个 PR 中提交了 https://github.com/taojy123/valval/pull/1 今天由于 merge 了这个 PR 代码又被改回去了，真是乌龙了 😣

xucs007 commented 4 years ago

最新的代码里没看到filepath:

ZSH ~/v/v/ grep filepath */.v vlib/compiler/compile_errors.v:83: fullpath := s.get_error_filepath() vlib/compiler/compile_errors.v:90: fullpath := s.get_error_filepath() vlib/compiler/compile_errors.v:93: // The filepath:line:col: format is the default C compiler vlib/compiler/compile_errors.v:147:fn (s &Scanner) get_error_filepath() string {

Changsen(John) Xu+86 180-1792-3348

------------------ Original ------------------ From: "taojy123";notifications@github.com; Send time: Saturday, Mar 14, 2020 1:15 AM To: "taojy123/valval"valval@noreply.github.com; Cc: "徐C.S."xucs007@qq.com; "Author"author@noreply.github.com; Subject: Re: [taojy123/valval] v 0.1.23 removed filepath module (#4)

不好意思，早上没仔细看你的 PR，其实不是 0.1.23 移除了 filepath，而是最新的 0.1.24 增加了 filepath.ext 来替代 os.ext，而 os.ext 即将被废弃。相关的更新之前在这个 PR 中提交了 #1 今天由于 merge 了这个 PR 代码又被改回去了，真是乌龙了 ?

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub, or unsubscribe.

taojy123 commented 4 years ago

源码里没有，但是确实可以 import filepath 0.1.24 的 CHANGELOG 中有提到 New filepath module https://github.com/vlang/v/blob/master/CHANGELOG.md

taojy123 commented 4 years ago

我又试了下现在 master 上还未发布的 0.1.25 版本的代码，好像又把 filepath 移除了 😂

valvalio / valval

v 0.1.23 removed filepath module #4