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

Add symlink for new IGListArrayUtilsInternal.m #1591

Closed
wants to merge 1 commit into from

Conversation

TimOliver
Copy link
Member

Changes in this pull request

I moved the functions from IGListArrayUtilsInternal.h to IGListArrayUtilsInternal.m to ensure the code was properly captured for code coverage, but I forgot to re-run the SPM script to generate a new symlink for it. This should fix the failing sample app build tests.

Checklist

  • All tests pass. Demo project builds and runs.
  • I added tests, an experiment, or detailed why my change isn't tested.
  • I added an entry to the CHANGELOG.md for any breaking changes, enhancements, or bug fixes.
  • I have reviewed the contributing guide

@facebook-github-bot
Copy link
Contributor

@TimOliver has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator.

@facebook-github-bot
Copy link
Contributor

@TimOliver merged this pull request in 2644442.

@TimOliver TimOliver deleted the fix-spm branch October 14, 2023 06:00
GreeMoz pushed a commit to appintheair/IGListKit that referenced this pull request Jan 31, 2024
Summary:
## Changes in this pull request

I moved the functions from ` IGListArrayUtilsInternal.h` to `IGListArrayUtilsInternal.m` to ensure the code was properly captured for code coverage, but I forgot to re-run the SPM script to generate a new symlink for it. This should fix the failing sample app build tests.

### Checklist

- [x] All tests pass. Demo project builds and runs.
- [x] I added tests, an experiment, or detailed why my change isn't tested.
- [x] I added an entry to the `CHANGELOG.md` for any breaking changes, enhancements, or bug fixes.
- [x] I have reviewed the [contributing guide](https://github.com/Instagram/IGListKit/blob/main/.github/CONTRIBUTING.md)

Pull Request resolved: Instagram#1591

Test Plan: Confirmed this lets the sample apps run again.

Reviewed By: fabiomassimo

Differential Revision: D50013194

Pulled By: TimOliver

fbshipit-source-id: ff4bebae781ebb460915458848cb340a89401a83
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants