← Back to team overview

yahoo-eng-team team mailing list archive

[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