chore(todo): remove todo tag

autoload/SpaceVim/layers/lang/json.vim

@@ -30,8 +30,6 @@
 " 3. enable_json5: Enable/Disable json5 support. Enabled by default.
 
 if exists('s:conceal')
-  " @bug s:conceal always return 0
-  "
   " because this script will be loaded twice. This is the feature of vim,
   " when call an autoload func, vim will try to load the script again
   finish

autoload/SpaceVim/plugins/a.vim

@@ -186,7 +186,7 @@ else
     call s:LOGGER.info('  >   type: ' . get(a:000, 0, 'alternate'))
     call s:LOGGER.info('  >  parse: ' . a:request_parse)
     call s:LOGGER.info('  > config: ' . a:conf_path)
-    " @question when should the cache be loaded?
+    " when should the cache be loaded?
     " if the local value s:project_config do not has the key a:conf_path
     " and the file a:conf_path has not been updated since last cache
     " and no request_parse specified
@@ -232,12 +232,11 @@ else
   function! s:parse(alt_config_json) abort
     call s:LOGGER.info('Start to parse alternate files for: ' . a:alt_config_json.root)
     let s:project_config[a:alt_config_json.root] = {}
-    " @question why need sory()
-    " if we have two key docs/*.md and docs/cn/*.md
+    " if we have two keys docs/*.md and docs/cn/*.md
     " with the first key, we can also find files in
     " docs/cn/ directory, for example docs/cn/index.md
     " and the alt file will be
-    " docs/cn/cn/index.md. this should be overrided by login in
+    " docs/cn/cn/index.md. this should be overrided by
     " docs/cn/*.md
     "
     " so we need to use sort, and make sure `docs/cn/*.md` is parsed after

autoload/SpaceVim/plugins/projectmanager.vim

@@ -195,8 +195,16 @@ else
     augroup END
   endif
   function! s:find_root_directory() abort
-    " @question confused about expand and fnamemodify
+    " confused about expand and fnamemodify
     " ref: https://github.com/vim/vim/issues/6793
+    " response from Bram Moolenaar
+    " When there is no file name there also isn't a directory in which that
+    " file exists.  Perhaps expand('%:p') should result in an error.
+    " Returning an empty string is basically the same as returning an error.
+    "
+    " fnamemodify() doesn't use the current file name.  Now what should it do
+    " with an empty string?  Not sure.  Anyway, the current behavoir is what
+    " users rely on, so let's not change it.
 
 
     " get the current path of buffer or working dir

syntax/SpaceVimTasksInfo.vim

@@ -7,7 +7,6 @@ syntax case ignore
 syn match TaskName /^\[.*\]/
 syn match TaskTitle /^Task\s\+Type\s\+Command/
 
-" @question Why \zs does not work in syntax file?
 " ref:
 " https://github.com/vim/vim/issues/598
 " https://stackoverflow.com/questions/49323753/vim-syntax-file-not-matching-with-zs