Document FXIOS-10335 Code comments

mattreaganmozilla commented 1 day ago

:scroll: Tickets

n/a (See below)

:bulb: Description

Comments only. No code changes.

:pencil: Checklist

You have to check all boxes before merging

[x] Filled in the above information (tickets numbers and description of your work)
[x] Updated the PR name to follow our PR naming guidelines
[ ] Wrote unit tests and/or ensured the tests suite is passing
[ ] When working on UI, I checked and implemented accessibility (minimum Dynamic Text and VoiceOver)
[ ] If needed, I updated documentation / comments for complex code and public methods
[ ] If needed, added a backport comment (example @Mergifyio backport release/v120)

mobiletest-ci-bot commented 1 day ago

	Messages
:book:	Project coverage: 33.03%
:book:	Edited 1 files
:book:	Created 0 files

Client.app: Coverage: 31.11

File	Coverage
BrowserViewController.swift	4.74%	⚠️

Generated by :no_entry_sign: Danger Swift against 04f37a7407d2e31cf3a80b4acdafd54f897b500f

mattreaganmozilla commented 1 day ago

cc @clarmso This PR has no code changes but several UI test failures, specifically in testAddNewAddressAllFieldsFilled and testUpdateAllAddressFields.

Those look like new tests just recently added. Would it be helpful to file a ticket to investigate further?

mozilla-mobile / firefox-ios