WordPress-iOS
WordPress-iOS copied to clipboard
wordpress-mobile
removed forced try & unwrap
Fixes # Safety checks: removed forced try & forced unwrap to avoid crash
| Warnings | |
|---|---|
| :warning: | PR is not assigned to a milestone. |
| :warning: | PR is missing at least one label. |
Generated by :no_entry_sign: dangerJS
![peril-wordpress-mobile[bot] avatar](https://avatars.githubusercontent.com/u/1306301?v=4 "Published by a pub.dev verified publisher"){kind=small}
@entunidotdeb just a note to say I "took over" your work in #19167 to 1) run all the CI checks on it (which don't run on external contributor PRs) and 2) add some additional tweaks.
I cherry picked your commit, so once merged you should appear in the list of contributors.
Also, once #19167 merges this PR should automatically be marked as merged, too.
Thank you so much for your help!
@mokagio thank you so much for informing. Just to confirm here, are CI checks supposed to run automatically on contributor's PR ? Happy to help more :)
Just to confirm here, are CI checks supposed to run automatically on contributor's PR ?
Not the full CI suite. For example, this PR has 2 successful checks:
But a maintainer PR has 20:
Ops, sorry, I didn't realize there was an open PR targeting my PR branch. But luckily Gio's PR got me covered. 😸
