~jslarraz/review-tools:use-container-in-review-classes

Branch merges

Propose for merging

Merged into review-tools:master at revision 4dde3f9bc9e65eabb972292c0bc99d100c5cfcc5

Alex Murray: Approve on 2024-06-05

Diff: 690 lines (+72/-112)

8 files modified

reviewtools/sr_common.py (+0/-12)
reviewtools/sr_functional.py (+8/-6)
reviewtools/sr_lint.py (+19/-25)
reviewtools/sr_security.py (+3/-2)
reviewtools/sr_tests.py (+0/-15)
reviewtools/tests/schemas/test_schema_snap.py (+0/-1)
reviewtools/tests/test_sr_functional.py (+7/-12)
reviewtools/tests/test_sr_lint.py (+35/-39)

api

Related source package recipes

No recipes using this branch. (?)

Create packaging recipe

Related snap packages

Related charm recipes

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

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

many: remove Review.pkg_filename and Review.unpack_dir in favor of Review.snap.filename and Review.snap.unpack_dir

ea60330... by Jorge Sancho Larraz on 2024-05-27

many: remove Review.pkg_files in favor of Review.snap.files

8e2cc2f... by Jorge Sancho Larraz on 2024-05-27

many: remove Review.pkg_bin_files in favor of Review.snap.bin_files

2979f04... by Jorge Sancho Larraz on 2024-06-02

rt/tests}/containers: improve coverage (missing extension and containerException)

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

1 → 2 of 2 results

First • Previous • Next • Last

8bf9b88... by Jorge Sancho Larraz on 2024-06-02

rt/tests}/containers: several improvements on exceptions handling

823fa93... by Jorge Sancho Larraz on 2024-06-02

rt/{,tests}/containers: move check_contaienr_format to BaseContainer initialization

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

tests/test.sh.expected: removed remaining "lint-snap-v2:assumes_valid"

15a3704... by Jorge Sancho Larraz on 2024-05-31

many: validate license and assumes via schema

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

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

1 → 2 of 2 results

First • Previous • Next • Last

7dcfef8... by Jorge Sancho Larraz on 2024-06-03

many: rename BaseContainer.get_yaml to get_file_as_yaml and add yaml_version and allow_duplicate_key arguments

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

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

1 → 2 of 2 results

First • Previous • Next • Last