Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add mypy support and fixup project to give no errors #512

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Commits on Jul 10, 2024

  1. Add mypy support and fixup project to give no errors

    * Permissive mypy configuration as starting point
    * Add minimal type annotations to get no mypy errors
    * Add runtime test for self.network before using the network
    * Network.add_node() doesn't accept LocalNode
    * PeriodicMessageTask.update() don't stop the task unless its running
    * Variable.desc ensure that the object is int
    * Variable.read() fail with ValueError unless a valid fmt is used
    * Variable.write() ensure the description is a string
    * BaseNode.__init__() fail if no node_id is provided
    * ObjectDictionary.__getitem__() when splitting "." only return if the object is not an ODVariable
    * ODRecord.__eq__(), ODArray.__eq__() and ODVariable.__eq__() test type of other before comparing
    * ODVariable.encode_raw(), .decode_phys(), .encode_phys() add type tests of ensure the input is of correct type
    * PdoMap various methods: ensure necessary attributes are set
    sveinse committed Jul 10, 2024
    Configuration menu
    Copy the full SHA
    3b66ae5 View commit details
    Browse the repository at this point in the history