lp:~jdstrand/click-reviewers-tools/refactor

Created by Jamie Strandboge and last modified
Get this branch:
bzr branch lp:~jdstrand/click-reviewers-tools/refactor
Only Jamie Strandboge can upload to this branch. If you are Jamie Strandboge please log in for upload directions.

Branch merges

Related bugs

Related blueprints

Branch information

Recent revisions

581. By Jamie Strandboge

clickreviews/cr_lint.py: don't run check_contents_for_hardcoded_paths() on
snap v2

580. By Jamie Strandboge

clickreviews/cr_security.py: clarify logic of package detection in two tests

579. By Jamie Strandboge

use self.is_click, etc instead of self._pkgfmt_version()
fix some 16.04isms in clickreviews/cr_security.py

578. By Jamie Strandboge

clickreviews/cr_lint.py:
- make sure md5sums test is run with clicks
- use self.is_click instead of 'self._pkgfmt_type() == "click"'
- only run check_pkgname() on click

577. By Jamie Strandboge

clickreviews/cr_tests.py: remove some 16.04 checking code

576. By Jamie Strandboge

remove a few more 16.04 checks

575. By Jamie Strandboge

update debian/changelog for refactor, etc

574. By Jamie Strandboge

remove some 16.04 checks and tests

573. By Jamie Strandboge

- rename bin/unpack-click as bin/unpack-package
- add common.py
  + refactor non-click and non-snap1 bits from cr_common.py
  + add proper package detection API and variables for click, snap1 and snap2
- clickreviews/cr_* (and tests)
  + adjust for common.py
  + use new detection variables and only run on click and snap1 snaps
  + cleanup cr_common.py to remove snap2 support (will be added in a later
    commit)
- clickreviews/cr_tests.py:
  + remove snap2 support (will be added in a later commit)
  + add detect_package support
  + various cleanups for refactoring
- adjust bin/*, clickreviews/* and clickreviews/tests/* to use common instead
  of cr_common
- bin/click-run-checks, clickreviews/modules.py: don't assume 'click' in
  variable names, etc
- pull existing snap v2 tests out of cr_*
- add bin/detect-package

572. By Jamie Strandboge

debian/changelog: update for last commit

Branch metadata

Branch format:
Branch format 7
Repository format:
Bazaar repository format 2a (needs bzr 1.16 or later)
Stacked on:
lp:click-reviewers-tools
This branch contains Public information 
Everyone can see this information.

Subscribers