Skip to content

[Draft] Revert "Add FXIOS-14673 [Technical Debt] [Redux] Add CopyWithUpdates … #32249

Draft
isabelrios wants to merge 1 commit intomainfrom
revert-FXIOS-14673-copywithupdate-macro
Draft

[Draft] Revert "Add FXIOS-14673 [Technical Debt] [Redux] Add CopyWithUpdates … #32249
isabelrios wants to merge 1 commit intomainfrom
revert-FXIOS-14673-copywithupdate-macro

Conversation

@isabelrios
Copy link
Contributor

…macro to more cleanly copy redux states with simple property changes (#31717)"

This reverts commit 419dce5.

💡 Description

This is just a Test to verify if the building time is increased by this PR

🎥 Demos

Before After
Demo

📝 Checklist

  • I filled in the ticket numbers and a description of my work
  • I updated the PR name to follow our PR naming guidelines
  • I ensured unit tests pass and wrote tests for new code
  • If working on UI, I checked and implemented accessibility (Dynamic Text and VoiceOver)
  • If adding telemetry, I read the data stewardship requirements and will request a data review
  • If adding or modifying strings, I read the guidelines and will request a string review from l10n
  • If needed, I updated documentation and added comments to complex code

…macro to more cleanly copy redux states with simple property changes (#31717)"

This reverts commit 419dce5.
@isabelrios isabelrios added the Do Not Merge ⛔️ This issue is a work in progress and is not ready to land label Feb 24, 2026
@mobiletest-ci-bot
Copy link

Messages
📖 Project coverage: 40.15%

💪 Quality guardian

1 tests files modified. You're a champion of test coverage! 🚀

🧟‍♂️ Monster PR

Wow, this PR is huge with 2786 lines changed!
Thanks for powering through such a big task 🙌.
Reviewers: feel free to ask for extra context, screenshots, or a breakdown to make reviewing smoother.

✅ New file code coverage

No new file detected so code coverage gate wasn't ran.

Client.app: Coverage: 38.26

File Coverage
SearchEngineSelectionState.swift 94.37%

Generated by 🚫 Danger Swift against 4bd4916

@mergify
Copy link
Contributor

mergify bot commented Feb 26, 2026

This pull request has conflicts when rebasing. Could you fix it @isabelrios? 🙏

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

Labels

Do Not Merge ⛔️ This issue is a work in progress and is not ready to land

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants