← Back to issue list

tox: pass CI environment variable

View original Github issue

Metadata

Project
starbase
Number
#102
Type
pull request
State
merged
Author
lengau
Labels
—
Created
Updated
Closed

Current evaluation

Merged as 3f7c6fa, configuring tox to pass the CI environment variable; upstreamed from canonical/craft-providers#328.

Resolution: closed resolved

Reason: PR #102 was merged into starbase by commit 3f7c6fa, adding the tox configuration needed to pass CI into test environments.

Evaluated by: mr-cal-bot

No scores available.

Issue body

- [x] Have you followed the guidelines for contributing? - [x] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)? - [x] Have you successfully run `tox`? ----- Upstreamed from https://github.com/canonical/craft-providers/pull/328

Evaluation history

Date Model Scores Action Summary
stealth/space-bunny-alpha — closed resolved Merged as 3f7c6fa, configuring tox to pass the CI environment variable; upstreamed from canonical/craft-providers#328.
stealth/space-bunny-alpha — closed resolved Merged canonical/craft-providers#328, upstreaming the tox change to pass the CI environment variable.
qwen/qwen3.6-35b-a3b — — Merged after approval by two reviewers and passing CI checks. The change upstreams a modification to pass the CI environment variable to tox, adding two lines to one file.
qwen3.6-35b-a3b-mtp-q6 — — Merged upstreamed changes to pass the CI environment variable in tox configurations. The pull request was integrated into the main branch after passing contribution checks.

Update history

No update history recorded yet.

Related work

Related issues

Issue Project State Summary Similarity
#295 ci: add tox craft-providers merged Merged PR #295 in commit 9ebe593, enabling tox for craft-providers CI and resolving CRAFT-1726.
80%
#742 fix(ci): install tox in ci for tics craft-providers closed Closed unmerged; superseded by craft-providers#745, merged as commit 2e31f55 to install tox in CI and restore TICS testing.
76%
#106 tox: improve UX starbase merged Merged PR #106 as commit 8b8fed8, improving tox UX and fixing #105; it ported UX changes from craft-providers PR #332.
72%
#576 build: fix tox test environments craft-parts merged Merged #576 as commit 576e2412, fixing craft-parts tox test environments with a one-file build configuration change (+3/-1).
72%
#1674 fix: make tox tests succeed charmcraft merged Merged in commit b24ccd5b: fixed the tox environment variable and added a workaround for canonical/operator#1212, allowing tox tests to succeed.
71%