remove excess comma

Bug #1477869 reported by LiangHuifei
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Ceilometer
Fix Released
Wishlist
LiangHuifei

Bug Description

File: ceilometer/tests/publisher/test_direct.py
Line:95 def test_direct_publisher(self, ):
The comma behind "self" is excess, so we should remove it.

Changed in ceilometer:
assignee: nobody → LiangHuifei (lianghuifei830)
status: New → In Progress
Revision history for this message
Julien Danjou (jdanjou) wrote :

Please do not open a bug for each commit you do. This is spam at this point. Thanks! :)

Changed in ceilometer:
importance: Undecided → Wishlist
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to ceilometer (master)

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

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to ceilometer (master)

Reviewed: https://review.openstack.org/205435
Committed: https://git.openstack.org/cgit/openstack/ceilometer/commit/?id=6c4f6771ca9978932afefc14fe1457279c8f1f1f
Submitter: Jenkins
Branch: master

commit 6c4f6771ca9978932afefc14fe1457279c8f1f1f
Author: lianghuifei <lianghuifei830@126.com>
Date: Fri Jul 24 16:02:31 2015 +0800

    Remove redundant comma

    The comma in def test_direct_publisher(self, ) is redundant,
    so we should remove it.

    Closes-Bug: #1477869
    Change-Id: I52707befa0ffd29658a39d28a93c7c24724cfe6a

Changed in ceilometer:
status: In Progress → Fix Committed
Thierry Carrez (ttx)
Changed in ceilometer:
milestone: none → liberty-2
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in ceilometer:
milestone: liberty-2 → 5.0.0
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.