yahoo-eng-team team mailing list archive
-
yahoo-eng-team team
-
Mailing list archive
-
Message #50465
[Bug 1579496] Re: inconsistent alias names for the lib imports
IMO, let's wait for some more time and get neutron-lib fully ready with
all the constant defined. We can address all the issues one shot and
avoid rework in the future.
** Changed in: neutron
Status: In Progress => Opinion
--
You received this bug notification because you are a member of Yahoo!
Engineering Team, which is subscribed to neutron.
https://bugs.launchpad.net/bugs/1579496
Title:
inconsistent alias names for the lib imports
Status in neutron:
Opinion
Bug description:
Currently, there is inconsistent alias names for the lib imports which
makes the code really difficult to read. Constants from neutron-lib is
being imported as consts / n_consts / lib_consts while constants from
neutron is also being imported as consts / n_consts. This is really
frustrating & confusing.
IMO, uniform alias will help to increase code readability and
maintainability.
FYI, we have already pointed out this while reviewing
https://review.openstack.org/#/c/299017/ and felt the need of uniform
alias (For details one can refer to the PS16 comment discussion)
I would suggest we can use a prefix 'n_' while importing from neutron
and 'nlib_' for neutron-lib imports.
To manage notifications about this bug go to:
https://bugs.launchpad.net/neutron/+bug/1579496/+subscriptions
References