Skip targets with unsupported platforms instead of throwing an error

[fortmarek]

Resolves #6030

Short description 📝

We should only skip targets in the PackageInfoMapper that have only unsupported platforms as a condition, such as linux.

How to test the changes locally 🧐

• You should be able to generate and build the app_with_spm_dependencies fixture. The error was caused by this SwiftLint target that has a condition with linux only

Contributor checklist ✅

• The code has been linted using run mise run lint:fix
• The change is tested via unit testing or acceptance testing, or both
• The title of the PR is formulated in a way that is usable as a changelog entry
• In case the PR introduces changes that affect users, the documentation has been updated

Reviewer checklist ✅

• The code architecture and patterns are consistent with the rest of the codebase
• Reviewer has checked that, if needed, the PR includes the label changelog:added, changelog:fixed, or changelog:changed, and the title is usable as a changelog entry

[waltflanagan]

This doesnt throw anymore, so we should be able to update this callsite and remove the do {} catch {} also

[fortmarek]

oh yeah, good point

[waltflanagan]

I'm not able to find where this ended up being able to throw out in a way that would end up failing things. Is the code that doesnt handle the throw in tuist cloud?

[fortmarek]

I'm not able to find where this ended up being able to throw out in a way that would end up failing things. Is the code that doesnt handle the throw in tuist cloud?

No, tuist generate would throw when mapping the PackageInfo from https://github.com/realm/SwiftLint. I'm actually now also not sure why the error was not caught but I feel like this refactor is fine.