Skip to content

Conversation

@AlexsanderDamaceno
Copy link

adds remove fixits (#85882) in order of remove redundant cases of try/unsafe expressions when that is diagnosed in some cases.

@AlexsanderDamaceno
Copy link
Author

@swift-ci please smoke test

@AlexsanderDamaceno
Copy link
Author

@swift-ci please test

@xedin
Copy link
Contributor

xedin commented Dec 11, 2025

Thank you for the contribution! I will take a look at the changes today and run CI for you if everything looks good.

@jamieQ
Copy link
Contributor

jamieQ commented Dec 11, 2025

FYI, not to be too 'lawyerly' about this, but someone else had already expressed interest in working on this task in the issue i opened (which is now assigned to them), so it would seem polite IMO to see if they are still interested in taking it on before progressing this. cc @robpatterson13. obviously still great to have people picking things up though!

@AlexsanderDamaceno
Copy link
Author

AlexsanderDamaceno commented Dec 11, 2025

FYI, not to be too 'lawyerly' about this, but someone else had already expressed interest in working on this task in the issue i opened (which is now assigned to them), so it would seem polite IMO to see if they are still interested in taking it on before progressing this. cc @robpatterson13. obviously still great to have people picking things up though!

Hey no problem, I saw now that @robpatterson13 just got assigned, @robpattterson13 feel free to take over this pr if desire to work on this.

@xedin
Copy link
Contributor

xedin commented Dec 11, 2025

The PR needs to have all of the existing test-cases updated to check for the new fix-it just like #85830 does for await. Please coordinate in the issue re: who is going to work on this since there are multiple people interested.

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.

3 participants