[Asterisk-code-review] alembic: Fix downgrade and tweak for sqlite (asterisk[master])

Kevin Harwell asteriskteam at digium.com
Wed Mar 2 12:26:24 CST 2016


Kevin Harwell has posted comments on this change.

Change subject: alembic: Fix downgrade and tweak for sqlite
......................................................................


Patch Set 1: Code-Review-1

(16 comments)

Found a few places with whitespace.

I also marked the superfluous "pass" keywords that were left in some of the files, but if you don't want to mess with removing them then feel free to  ignore them since they are not actually hurting anything.

https://gerrit.asterisk.org/#/c/2328/1/contrib/ast-db-manage/config/versions/3bcc0b5bc2c9_add_allow_reload_to_ps_transports.py
File contrib/ast-db-manage/config/versions/3bcc0b5bc2c9_add_allow_reload_to_ps_transports.py:

Line 23:     pass
While you are making changes the "pass" can be dropped


Line 28:     pass
Another one to drop


https://gerrit.asterisk.org/#/c/2328/1/contrib/ast-db-manage/config/versions/423f34ad36e2_fix_pjsip_qualify_ti.py
File contrib/ast-db-manage/config/versions/423f34ad36e2_fix_pjsip_qualify_ti.py:

Line 22:     pass
Another "pass" that could be dropped


Line 30:     pass
remove "pass"


https://gerrit.asterisk.org/#/c/2328/1/contrib/ast-db-manage/config/versions/43956d550a44_add_tables_for_pjsip.py
File contrib/ast-db-manage/config/versions/43956d550a44_add_tables_for_pjsip.py:

Line 186:     
Extra space/tab/red blob


Line 201:         for e in enums:    
Another red blob


https://gerrit.asterisk.org/#/c/2328/1/contrib/ast-db-manage/config/versions/461d7d691209_add_pjsip_qualify_timeout.py
File contrib/ast-db-manage/config/versions/461d7d691209_add_pjsip_qualify_timeout.py:

Line 19:     pass
Can remove the "pass"


Line 27:     pass
Found another "pass" that could be removed.


https://gerrit.asterisk.org/#/c/2328/1/contrib/ast-db-manage/config/versions/4c573e7135bd_fix_tos_field_types.py
File contrib/ast-db-manage/config/versions/4c573e7135bd_fix_tos_field_types.py:

Line 40:         
More whitespace


Line 41:     pass
extra "pass"


Line 65:     pass
another "pass"


https://gerrit.asterisk.org/#/c/2328/1/contrib/ast-db-manage/config/versions/4da0c5f79a9c_create_tables.py
File contrib/ast-db-manage/config/versions/4da0c5f79a9c_create_tables.py:

Line 327:     
Whitespace


Line 333:     
        :     
red blobs


Line 342:         for e in enums:    
Another whitespace


https://gerrit.asterisk.org/#/c/2328/1/contrib/ast-db-manage/config/versions/dbc44d5a908_add_missing_columns_to_sys_and_reg.py
File contrib/ast-db-manage/config/versions/dbc44d5a908_add_missing_columns_to_sys_and_reg.py:

Line 30:     pass
Another "pass" if removing them


Line 39:     pass
"pass" again


-- 
To view, visit https://gerrit.asterisk.org/2328
To unsubscribe, visit https://gerrit.asterisk.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I862b0739eb3fd45ec3412dcc13c2340e1b7baef8
Gerrit-PatchSet: 1
Gerrit-Project: asterisk
Gerrit-Branch: master
Gerrit-Owner: George Joseph <george.joseph at fairview5.com>
Gerrit-Reviewer: Anonymous Coward #1000019
Gerrit-Reviewer: Kevin Harwell <kharwell at digium.com>
Gerrit-HasComments: Yes



More information about the asterisk-code-review mailing list