| Commit message (Collapse) | Author | Age | Files | Lines |
|---|
| |
|
|
| |
Change-Id: I580b2c0b726b161821507e7198324c4c10e8a8d9
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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
|
