stlehmann / pyads

Python wrapper for TwinCAT ADS
MIT License
254 stars 94 forks source link

moved write_list_by_name underneath write_by name #290

Closed chrisbeardy closed 2 years ago

chrisbeardy commented 2 years ago

just a tidy up, so all read_by_name methods are first then the write_by_names methods as opposed to jumbling them up

coveralls commented 2 years ago

Pull Request Test Coverage Report for Build 1504085023


Files with Coverage Reduction New Missed Lines %
pyads/testserver/testserver.py 5 90.76%
<!-- Total: 5 -->
Totals Coverage Status
Change from base Build 1475440781: -0.3%
Covered Lines: 1665
Relevant Lines: 1768

💛 - Coveralls
stlehmann commented 2 years ago

We can do that. Brings more shape to the codebase. Don't know why the coverage decreased but can see no issues here. I'll merge it anyway.