MXParallaxHeader icon indicating copy to clipboard operation
MXParallaxHeader copied to clipboard

Fix SPM Usage in Xcode 12.5+

Open rogerluan opened this issue 4 years ago • 12 comments

Description

On Xcode 12.5, apparently the umbrella header generation logic changed, so this stopped working on my project because my project couldn't find MXScrollView anymore, when importing this lib using SPM. The problem was that the generated umbrella header didn't contain references to all the public headers. This PR aims to fix that.

Motivation and Context

How Has This Been Tested?

It compiles on my project now 😄

Before:

import MXParallaxHeader
…
let scrollView = MXScrollView // Couldn't find reference to MXScrollView

Now, the same code above compiles just fine.

Types of changes

  • [x] Bug fix (non-breaking change which fixes an issue)
  • [ ] New feature (non-breaking change which adds functionality)
  • [ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist:

  • [x] My code follows the code style of this project.
  • [ ] My change requires a change to the documentation. - Nope
  • [ ] I have updated the documentation accordingly. - There's no need to update anything

rogerluan avatar Sep 14 '21 02:09 rogerluan

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

stale[bot] avatar Jan 03 '22 21:01 stale[bot]

This is a PR, it's not stale 😢

rogerluan avatar Jan 05 '22 19:01 rogerluan

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

stale[bot] avatar Apr 17 '22 05:04 stale[bot]

Still valid

rogerluan avatar Apr 18 '22 19:04 rogerluan

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

stale[bot] avatar Jul 10 '22 11:07 stale[bot]

Still valid.

rogerluan avatar Jul 10 '22 12:07 rogerluan

@maxep could this PR be reviewed any time? 🙇

rogerluan avatar Jul 10 '22 12:07 rogerluan

@maxep hi!👋 feel free to add me as a contributor! Happy to move things forward :) Just make sure I have enough permissions to publish releases (and maybe pods?), so I don't have to bother you again 😄 cheers!

rogerluan avatar Jul 11 '22 20:07 rogerluan

@rogerluan, super nice, thank you!

So I've added you as a collaborator on this project. I've tried to add your email (from your GH profile) to cocoapods, but it seems that you are not yet registered. Could you please register? 👉 https://guides.cocoapods.org/making/getting-setup-with-trunk.html#getting-started

Thanks again!

maxep avatar Jul 13 '22 16:07 maxep

I'll do that tomorrow! 🤗

rogerluan avatar Jul 13 '22 21:07 rogerluan

Done @maxep ! :) Same email as my GH profile

rogerluan avatar Jul 15 '22 09:07 rogerluan

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

stale[bot] avatar Sep 21 '22 04:09 stale[bot]