OpenCyphal / pycyphal

Python implementation of the Cyphal protocol stack.
https://pycyphal.readthedocs.io/
MIT License
119 stars 106 forks source link

Remove CLI #139

Closed pavel-kirienko closed 3 years ago

pavel-kirienko commented 3 years ago

Fixes #134

The new repository for the CLI is being worked on, stay tuned. This PR should be merged once it is ready.

The integration test for the demo application had to be rewritten because the old one was built on top of the CLI.

The overall directory layout has been altered: the demo script and the DSDL namespaces it depends on are now located directly under the project root at /demo/. This should make the demo more accessible.

The coverage scaffolding was changed: now we use coverage run ... directly instead of relying on sitecustomize.py. This is simpler and easier to maintain.

The integration test suite does no longer generate DSDL packages in the project root (I've been looking for an easy way to fix it for a while).

This is the last substantial changeset before the v1.1 release.

coveralls commented 3 years ago

Pull Request Test Coverage Report for Build 37052464


Changes Missing Coverage Covered Lines Changed/Added Lines %
tests/demo/_subprocess.py 3 4 75.0%
pyuavcan/application/heartbeat_publisher.py 21 25 84.0%
tests/demo/_demo_app.py 180 184 97.83%
<!-- Total: 341 350 97.43% -->
Files with Coverage Reduction New Missed Lines %
pyuavcan/presentation/_port/_publisher.py 1 91.27%
pyuavcan/presentation/_presentation.py 1 93.33%
pyuavcan/transport/can/media/socketcan/_socketcan.py 1 83.41%
pyuavcan/transport/can/_session/_input.py 1 98.56%
pyuavcan/presentation/_port/_client.py 2 79.4%
pyuavcan/transport/redundant/_session/_input.py 2 97.38%
pyuavcan/transport/udp/_session/_input.py 2 83.78%
pyuavcan/presentation/_port/_base.py 3 90.91%
pyuavcan/application/heartbeat_publisher.py 4 88.89%
pyuavcan/presentation/_port/_subscriber.py 6 80.6%
<!-- Total: 23 -->
Totals Coverage Status
Change from base Build 36965372: 0.0002%
Covered Lines: 11055
Relevant Lines: 11407

💛 - Coveralls
pavel-kirienko commented 3 years ago

Done, the new tool is available for use. Read the announcement here: https://forum.uavcan.org/t/yakut-the-new-name-for-pyuavcan-cli-tool/1049