search

neovim / pynvim

Python client and plugin host for Nvim
http://pynvim.readthedocs.io/en/latest/
Apache License 2.0
1.48k stars 118 forks source link

coveragerc: parallel=1 #421

Closed blueyed closed 4 years ago

blueyed commented 4 years ago

Fixes:

INTERNALERROR> coverage.misc.CoverageException: Couldn't use data file '…/.coverage': UNIQUE constraint failed: meta.key

Ref: https://github.com/neovim/pynvim/pull/418#issuecomment-572617071

blueyed commented 4 years ago

Scrutinizer complains about missing coverage:

Code Coverage was not sent to Scrutinizer. Make sure that you have set-up your external service correctly. You can find more information on this in the documentation:

https://scrutinizer-ci.com/docs/tools/external-code-coverage/

I guess that's what ocular is meant to do? There's no error in logs currently, but needs investigation then.

justinmk commented 4 years ago

Thanks for trying :)

This is better than having all the travis builds fail, so let's merge it?

justinmk commented 4 years ago

We could also remove scrutinizer, is it ever useful?

blueyed commented 4 years ago

We could also remove scrutinizer, is it ever useful?

Ok with me. /cc @fwalch (via bfa672e)

I've checked that the number of runs matches our uploads / calls of ocular: https://github.com/neovim/python-client/blob/97ed85a8ca2deaa1689c0dd5b979a2d3ae72b716/.scrutinizer.yml#L131-L134.

blueyed commented 4 years ago

Removed it. Hopefully it stops being triggered once merged, otherwise somebody would need to disable it (https://scrutinizer-ci.com/g/neovim/pynvim/).