Skip to content

Conversation

crazytonyli
Copy link
Contributor

Nothing interesting. It's okay to merge after CI is green.

Regression Notes

PR submission checklist:

  • I have completed the Regression Notes.
  • I have considered adding unit tests for my changes.
  • I have considered adding accessibility improvements for my changes.
  • I have considered if this change warrants user-facing release notes and have added them to RELEASE-NOTES.txt if necessary.

Testing checklist: N/A

@crazytonyli crazytonyli added this to the 24.3 milestone Feb 15, 2024
@crazytonyli crazytonyli requested a review from mokagio February 15, 2024 00:25
@crazytonyli crazytonyli self-assigned this Feb 15, 2024
@mokagio
Copy link
Contributor

mokagio commented Feb 19, 2024

CI failed because apparently the Gutenberg VC requires access to Alamofire:

image

I'm going to push this PR from 24.3 to 24.4 because it's just an import removal so we aren't in any rush to merge it. This will allow me to focus on #22612 in the hope of slotting that more substantial change in.

@mokagio mokagio modified the milestones: 24.3, 24.4 Feb 19, 2024
@crazytonyli crazytonyli marked this pull request as draft February 19, 2024 06:58
auto-merge was automatically disabled February 19, 2024 06:58

Pull request was converted to draft

@crazytonyli crazytonyli deleted the remove-unused-imports branch February 21, 2024 00:24
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