The contents of this repository are still available in the Git source
code management system. To see the contents of this repository before it
reached its end of life, please check out the previous commit with
"git checkout HEAD^1".
For any further questions, please email
openstack-discuss@lists.openstack.org or join #openstack-telemetry on OFTC.
Change-Id: Ic8a7001e708d8da7cf8951eefd0a96762ea5fa46
It is an initial seed of hacking rules based on neutron and
nova projects.
For ceilometer it prevents bug 1508442 and closes/prevents
bug 1529836.
All the requirements about the hacking rules were corrected
on this propose review.
To be clear about the requirements, the only alarmed was
about import[1], in specific H306
[H306] "Alphabetically order your imports by the full module
path. Organize your imports"
[1] http://docs.openstack.org/developer/hacking/
Change-Id: Ia530873078c97925bfb91fde87b38131426a74c2
Closes-Bug: 1529836
oslo libraries once have issue on its namespace, but they all have
migrated to the right one during liberty cycle. It is time to drop
temporary local checks for this issue. Since this is the only
local check we have, this patch entirely removes all related files.
Change-Id: I8fe7a59f51ee8af25e3dade70da645cd646fa0b4
Currently, oslo team starts to drop old oslo namespace, the old import
style can work now, but not any more in next development cycle. We've
already done and will continue do the migrate jobs, but we need a way
to prevent accidently import such issue again. The hacking rules cannot
satisfy the need, so let's do the local check.
Code are basically stolen from Nova project.
Note, this patch also fixes some new introduced oslo namespace issues.
Change-Id: I98c81c47a727ed71fb75820f21276cb989d47714
Matthias Runge