mv-python

Results 197 comments of mv-python

@da-woods @scoder humble ping. Is there anything blocking accepting this PR?

Thank you @da-woods for your review. I have implemented all your comments.

> Just for my understanding, it's creating the links with this configuration option? I am not sphinx expert but yes. This is triggering creation of links to python (C links...

BTW the PR is not migrating all mentions of Python/C API. There are some tables (e.g. [1]) which needs to be migrated, but maybe more refactoring should go there so...

> what adds the most fat, what can an author of a cython module do to slim it down? See: * https://cython.readthedocs.io/en/latest/src/userguide/faq.html#how-do-i-reduce-the-size-of-the-binary-modules * https://github.com/cython/cython/issues/6045 * https://github.com/cython/cython/issues/4425#issuecomment-1966330797 * https://github.com/cython/cython/issues/6051

> 2) it's missing any kind of testing Shadow file is already tested in: `Cython/Tests/TestShadow.py`. Do we need additional tests since this PR is adding just symbolic links?

OK I did some investigation and the root cause is following: During calling `CClassScope.declare_cfunction()` the `entries` are search whether already exit for given function/method: https://github.com/cython/cython/blob/a78d102201a4105dea4d129ab391e7dfb6af2807/Cython/Compiler/Symtab.py#L2501 * For "normal" methods it...

@1605200517 for now there is no such docs available. You need to read consul docs: https://www.consul.io/intro/getting-started/install.html and after you should be able to map python-consul API to consul API -...

I prefer option with one additional release with warnings. I have quickly reviewed the proposed PR and see my comments. Regarding tornado implementation, here should be code snippet for support...

I have rerun tests and now are green. We have an issue in test suite.... I have created new PR #220 which should fix the issues with unittests. I have...