Skip to content

Conversation

@ahal
Copy link
Contributor

@ahal ahal commented Jun 13, 2025

No description provided.

@ahal ahal self-assigned this Jun 13, 2025
@ahal ahal force-pushed the ahal/push-xmmvrouvmmls branch 2 times, most recently from e64a862 to ee08b8c Compare June 13, 2025 18:59
headers={
"Accept": "application/vnd.github+json",
"Authorization": "Bearer abc",
},
Copy link
Contributor Author

Choose a reason for hiding this comment

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

I'm not sure what's going on here. At first I was able to reproduce this locally, and then I did something and now I can't anymore, but it still reproduces in CI?

I guess I could edit the test to be less strict about what it asserts, but I'm worried this might be uncovering an actual issue :/

Copy link
Contributor

Choose a reason for hiding this comment

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

Do you mind adding something in the commit message about this test change?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Will do.

Turns out that locally I had somehow managed to re-install the old dependency set. So it's at least consistent. I guess it's a change to aiohttp.

@ahal ahal requested a review from a team June 17, 2025 13:40
The test change is needed to fix a failure that started happening with
this dependency bump. I didn't track the change down, but likely
involves the aiohttp update.
@ahal ahal force-pushed the ahal/push-xmmvrouvmmls branch from ee08b8c to b41168e Compare June 17, 2025 18:18
@ahal ahal requested a review from jcristau June 17, 2025 18:18
@ahal ahal merged commit 0e2438a into mozilla-releng:main Jun 18, 2025
10 checks passed
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.

2 participants