[Github-comments] [geany/geany] Add Ocaml ctags parser (PR #3163)

[elextr]

> Oh, or do you mean how it works in universal-ctags? 

Yes

> There the unit tests work differently and preserve the name of the tested file so the output is always the same.

Ok, I thought there were other languages that required modules to be in files of the same name, but maybe ctags doesn't parse modules for those. 

>  with our unit tests the source file name is a temporary name

Where does that happen?  AFAICT `runner.sh` doesn't copy the source file, it just puts the tagfile in a temporary directory?

Or do you mean that by convention our test file names are chosen to reflect the test purpose (hence your choice of `simple.ml` for the first test file)?  If so thats just a convention, and if it clashes with the language requirements it can be changed and your test file called `modulefoo.ml`.

Personally I would have disabled the tests not the parsing.

-- 
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany/pull/3163#issuecomment-1099739446
You are receiving this because you are subscribed to this thread.

Message ID: <geany/geany/pull/3163/c1099739446 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.geany.org/pipermail/github-comments/attachments/20220414/64508ac2/attachment.htm>