Skip to content

ci: add integration tests that run with lowest and highest dependency…#897

Open
ahal wants to merge 1 commit intotaskcluster:mainfrom
ahal:ahal/push-yuzmlxryuyxv
Open

ci: add integration tests that run with lowest and highest dependency…#897
ahal wants to merge 1 commit intotaskcluster:mainfrom
ahal:ahal/push-yuzmlxryuyxv

Conversation

@ahal
Copy link
Collaborator

@ahal ahal commented Feb 4, 2026

… resolution

@ahal ahal force-pushed the ahal/push-yuzmlxryuyxv branch 9 times, most recently from 0704c1f to 1df45be Compare February 5, 2026 20:05
@ahal ahal self-assigned this Feb 5, 2026
@ahal ahal marked this pull request as ready for review February 5, 2026 20:05
@ahal ahal requested a review from a team as a code owner February 5, 2026 20:05
@ahal ahal requested a review from abhishekmadan30 February 5, 2026 20:05
@ahal ahal marked this pull request as draft February 5, 2026 20:09
@ahal ahal force-pushed the ahal/push-yuzmlxryuyxv branch from 1df45be to 7d30d3a Compare February 5, 2026 20:11
@ahal ahal marked this pull request as ready for review February 5, 2026 20:11
@ahal ahal enabled auto-merge (rebase) February 5, 2026 20:12
@jcristau jcristau disabled auto-merge February 6, 2026 09:14
Comment on lines +25 to +26
worker:
taskcluster-proxy: true
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm surprised this enables the proxy, I wouldn't have expected it to be needed (I know you're just moving this code here, I just hadn't noticed this before).

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Honestly I didn't notice either. @bhearsum was this intentional?

I'll leave it in this PR, we can attempt disabling in a follow-up

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I have recollection of needing it. It could be copy/paste failure, or something I failed to document. I've opened #900 to remove it, and we'll see if that passes tests.

"cookiecutter~=2.1",
"json-e>=2.7",
"mozilla-repo-urls",
"mozilla-repo-urls>=0.1.1",
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is the last version that supports Python 3.9 and 3.10..

This might bite us eventually if we ever need to depend on a new change there.

"sphinx-book-theme >=1.1.4",
"sphinx-taskgraph>=0.1.0",
"sphinxcontrib-mermaid>=1.0.0",
"zstandard>=0.25.0",
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I just put a minimum bound to the maximum version that still support 3.9 for these. They're just dev dependencies, so no need to keep a wide constraint.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants