RFE: Add initial static type hint checking support

Bug #1633756 reported by John L. Villalovos
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
ironic-lib
Triaged
Wishlist
Unassigned

Bug Description

Add initial support for using static type hint checking.

Using the tool mypy:
http://mypy-lang.org/

This will require updating the tox.ini to run mypy in the [pep8] section.

And adding type hints into the code, at least a little amount to get things going.

tags: added: rfe
summary: - Add static type hint checking support
+ RFE: Add static type hint checking support
Changed in ironic-lib:
assignee: nobody → John L. Villalovos (happycamp)
summary: - RFE: Add static type hint checking support
+ RFE: Add initial static type hint checking support
description: updated
Changed in ironic-lib:
status: New → In Progress
Changed in ironic:
assignee: nobody → John L. Villalovos (happycamp)
Ruby Loo (rloo)
Changed in ironic:
importance: Undecided → Wishlist
Changed in ironic-lib:
importance: Undecided → Wishlist
Revision history for this message
Jim Rollenhagen (jim-rollenhagen) wrote :

Approved for ironic-lib in today's meeting, leaving tag as 'rfe' since it isn't approved for ironic proper.

no longer affects: ironic
Revision history for this message
Ruby Loo (rloo) wrote :

This was discussed in today's ironic meeting [1]. It wasn't clear whether this would be useful or not, and if useful, would it mean that we'd require all future code to support this?

We decided that it would be OK to try this out with ironic-lib. Depending on how that goes, we'll decide about whether to use it for other ironic projects, etc.

This RFE doesn't apply to ironic any more, just ironic-lib, so I applied the rfe-approved tag.

[1] starting at 17:33:53, http://eavesdrop.openstack.org/meetings/ironic/2016/ironic.2016-11-28-17.00.log.html

tags: added: rfe-approved
removed: rfe
Dmitry Tantsur (divius)
Changed in ironic-lib:
status: In Progress → Triaged
assignee: John L. Villalovos (happycamp) → nobody
tags: added: low-hanging-fruit
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.