Skip to content

Conversation

@TimOliver
Copy link
Member

Changes in this pull request

It's been a few years since I did an audit of our test suite. The majority of changes to IGListKit since then were adding additional error checking and hardening, so this was relatively straightforward.

Checklist

  • All tests pass. Demo project builds and runs.
  • I added tests, an experiment, or detailed why my change isn't tested.
  • I have reviewed the contributing guide

@meta-cla meta-cla bot added the CLA Signed label Jan 27, 2026
@github-actions
Copy link

github-actions bot commented Jan 27, 2026

1 Error
🚫 Any source code changes should have an entry in CHANGELOG.md.
3 Warnings
⚠️ Big PR
⚠️ All pull requests should have a milestone attached, unless marked #trivial.
⚠️ Examples/Examples-iOS/IGListKitExamples-UITests/DemosViewControllerUITests.swift#L89 - Variable name 'app' should be between 4 and 40 characters long
identifier_name DemosViewControllerUITests.swift:89

Generated by 🚫 Danger

@meta-codesync
Copy link

meta-codesync bot commented Jan 27, 2026

@TimOliver has imported this pull request. If you are a Meta employee, you can view this in D91551666.

@facebook-github-bot
Copy link
Contributor

@TimOliver has updated the pull request. You must reimport the pull request before landing.

1 similar comment
@facebook-github-bot
Copy link
Contributor

@TimOliver has updated the pull request. You must reimport the pull request before landing.

@facebook-github-bot
Copy link
Contributor

@TimOliver has updated the pull request. You must reimport the pull request before landing.

@facebook-github-bot
Copy link
Contributor

@TimOliver has updated the pull request. You must reimport the pull request before landing.

@meta-codesync
Copy link

meta-codesync bot commented Jan 28, 2026

@TimOliver merged this pull request in 062b2f9.

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.

3 participants