Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Create a Makefile target simulating building for Library Evolution; address issues with Xcode 16 running it #199

Conversation

gemmakbarlow
Copy link
Contributor

@gemmakbarlow gemmakbarlow commented Sep 23, 2024

  • Add build-for-library-evolution target to the Makefile. This replicates an issue I'm seeing in our project building with Xcode 16, i.e. -

image

  • Updates some dependencies to address various errors outside of this repository affecting the build-for-library-evolution target
  • Adds @unknown default to one switch statement inside this project causing some issues

Happy to add CI around this target if desired; let me know what you think!

@stephencelis
Copy link
Member

@gemmakbarlow I think CI would be a good idea. Want to add a job to the ci.yml file?

@gemmakbarlow
Copy link
Contributor Author

Sounds good, will do in a few hours.

@gemmakbarlow gemmakbarlow force-pushed the gemmakbarlow/xcode-16-library-evolution-issues branch from 647033d to eb42355 Compare September 24, 2024 01:38
@gemmakbarlow
Copy link
Contributor Author

gemmakbarlow commented Sep 24, 2024

Sounds good, will do in a few hours.

@stephencelis - Done, and all is green. See what you think, feedback welcomed.

Copy link
Member

@stephencelis stephencelis left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@stephencelis stephencelis merged commit 7ada838 into pointfreeco:main Sep 25, 2024
7 checks passed
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.

2 participants