~jslarraz/review-tools:use-logging-module

Branch merges

Propose for merging

Merged into review-tools:master at revision f2c630c4f9025a91435ee9a60d93b382756c13e4

Alex Murray: Approve on 2024-06-06

Diff: 1149 lines (+150/-212)

17 files modified

bin/create-snap-declaration (+3/-3)
bin/dump-tool (+9/-9)
bin/rock-check-notices (+6/-7)
bin/rock-updates-available (+7/-7)
bin/snap-check-notices (+6/-7)
bin/snap-review (+8/-18)
bin/snap-updates-available (+10/-10)
bin/snap-verify-declaration (+8/-11)
bin/store-query (+11/-9)
reviewtools/__init__.py (+16/-0)
reviewtools/available.py (+5/-7)
reviewtools/common.py (+17/-58)
reviewtools/sr_declaration.py (+4/-3)
reviewtools/store.py (+26/-27)
tests/test-dump-tool.sh.expected (+8/-8)
tests/test-snap-verify-declaration.sh.expected (+3/-25)
tests/test-updates-available.sh.expected (+3/-3)

api

Related source package recipes

No recipes using this branch. (?)

Create packaging recipe

Related snap packages

Related charm recipes

f2c630c... by Jorge Sancho Larraz on 2024-05-31

many: use logging module instead of custom functions in cli applications

MR https://code.launchpad.net/~jslarraz/review-tools/+git/review-tools/+merge/466961

	test:0 (build)
	coverage:0 (build)

1 → 2 of 2 results

First • Previous • Next • Last

405f910... by Evan Caville on 2024-06-06

Merge remote-tracking branch 'evancaville/overrides-update'

7de878d... by Jorge Sancho Larraz on 2024-06-05

rt/sr_common,tests/test.sh.expected: remove common.error in check classes in favor of exceptions. Uncaught exceptions will be later interpreted as runtime errors by snap-review

	test:0 (build)
	coverage:0 (build)

1 → 2 of 2 results

First • Previous • Next • Last

72ef927... by Jorge Sancho Larraz on 2024-06-04

many: validate confinement and grade attributes via schema

	test:0 (build)
	coverage:0 (build)

1 → 2 of 2 results

First • Previous • Next • Last

b100807... by Jorge Sancho Larraz on 2024-06-04

bin/snap-review,rt/sr_lint: enable --allow-gadget argument for snap-review

MR: https://code.launchpad.net/~jslarraz/review-tools/+git/review-tools/+merge/466884

	test:0 (build)
	coverage:0 (build)

1 → 2 of 2 results

First • Previous • Next • Last

995ce42... by Jorge Sancho Larraz on 2024-05-31

many: run check-names during unittest

MR: https://code.launchpad.net/~jslarraz/review-tools/+git/review-tools/+merge/466731

	test:0 (build)
	coverage:0 (build)

1 → 2 of 2 results

First • Previous • Next • Last

292ed06... by Jorge Sancho Larraz on 2024-06-04

many: remove individual bin/snap-check-*

MR: https://code.launchpad.net/~jslarraz/review-tools/+git/review-tools/+merge/466859

	test:0 (build)
	coverage:0 (build)

1 → 2 of 2 results

First • Previous • Next • Last

220620a... by Jorge Sancho Larraz on 2024-06-04

rt/c/squashfs_container: use recursive_rm instead of shutil.rmtree

MR: https://code.launchpad.net/~jslarraz/review-tools/+git/review-tools/+merge/466848

	test:0 (build)
	coverage:0 (build)

1 → 2 of 2 results

First • Previous • Next • Last

db4a16a... by Jorge Sancho Larraz on 2024-06-04

bin/snap-review: add runtime errors to report, so we can fail the review

4dde3f9... by Jorge Sancho Larraz on 2024-06-04

many: remove Review.tmp_dir in favor of Review.snap.tmp_dir

MR: https://code.launchpad.net/~jslarraz/review-tools/+git/review-tools/+merge/466877

	test:0 (build)
	coverage:0 (build)

1 → 2 of 2 results

First • Previous • Next • Last