[ 
https://issues.apache.org/jira/browse/BEAM-8572?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16968716#comment-16968716
 ] 

Udi Meiri commented on BEAM-8572:
---------------------------------

Ideas:
1a. Migrate precommit unit testing to pytest, mark Cython requiring tests as 
such (pytest.mark.cython), and only run them when 
1b. Same migration, but add a flag to conftest.py that makes it verify that 
cythonize has been run. (utils.check_compiled('apache_beam.coders')?)

> tox environment: assert on Cython source file presence
> ------------------------------------------------------
>
>                 Key: BEAM-8572
>                 URL: https://issues.apache.org/jira/browse/BEAM-8572
>             Project: Beam
>          Issue Type: Bug
>          Components: sdk-py-core, testing
>            Reporter: Udi Meiri
>            Assignee: Udi Meiri
>            Priority: Major
>
> Add an assertion somewhere that checks if Cythonized files are present in the 
> sdist tarball in use. That is for "tox -e py27" assert that these files are 
> not present, for "tox -e py27-cython" assert that they are present.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

Reply via email to