Rally

Consider using 'dt' as alias for datetime imports

Bug #1535786 reported by Illia Khudoshyn
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Rally
Fix Released
Wishlist
li,chen

Bug Description

tests/unit/cli/commands/test_task.py
tests/unit/cli/commands/test_verify.py

li,chen (chen-li)
Changed in rally:
assignee: nobody → li,chen (chen-li)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to rally (master)

Fix proposed to branch: master
Review: https://review.openstack.org/269958

Changed in rally:
status: New → In Progress
Andriy Kurilin (andreykurilin)
Changed in rally:
importance: Undecided → Wishlist
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to rally (master)

Reviewed: https://review.openstack.org/269958
Committed: https://git.openstack.org/cgit/openstack/rally/commit/?id=e34c6c9a5efc042be2241cf34e0dbb4e4ce75f7b
Submitter: Jenkins
Branch: master

commit e34c6c9a5efc042be2241cf34e0dbb4e4ce75f7b
Author: chen-li <email address hidden>
Date: Wed Jan 20 17:49:15 2016 +0800

    Using 'dt' as alias for datetime imports

    Many well-known libraries have a conventions regarding which
    alias to use. Choosing a different one will not break the code
    but makes it less readable and harder to maintain for other
    developers. Unless there is specific reason, it is usually best
    to use the recommended alias which most people are already
    familiar with. For datetime this convention is dt.

    Change-Id: Ifba7b7beafdac8377b98ce952b7d3b9dade01670
    Closes-Bug: #1535786

Changed in rally:
status: In Progress → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.
Subscribing...

Other bug subscribers

Remote bug watches

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