Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix(f3): use the correct context in tests #12582

Merged
merged 1 commit into from
Oct 11, 2024
Merged

Conversation

Stebalien
Copy link
Member

@Stebalien Stebalien commented Oct 11, 2024

Checklist

Before you mark the PR ready for review, please make sure that:

@Stebalien Stebalien added the skip/changelog This change does not require CHANGELOG.md update label Oct 11, 2024
@Stebalien Stebalien changed the title fix(f3): use the correct manifest in tests fix(f3): use the correct context in tests Oct 11, 2024
@Stebalien Stebalien force-pushed the steb/f3-correct-manifest branch from 37c31e6 to 4ecf77b Compare October 11, 2024 00:15
@Stebalien Stebalien requested a review from rvagg October 11, 2024 00:16
@Stebalien
Copy link
Member Author

The F3 test failure isn't related to this change.

@Stebalien Stebalien merged commit ba17b9a into master Oct 11, 2024
84 of 85 checks passed
@Stebalien Stebalien deleted the steb/f3-correct-manifest branch October 11, 2024 02:41
@BigLep
Copy link
Member

BigLep commented Oct 13, 2024

@Stebalien : does this need to be backported for 1.30.0?

@Stebalien
Copy link
Member Author

Not this one, no. But I do need to go through everything else on master because we have quite a few F3 PRs...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
skip/changelog This change does not require CHANGELOG.md update
Projects
Status: ☑️ Done (Archive)
Development

Successfully merging this pull request may close these issues.

3 participants