These were quite hard to track down since the errors that would occur would often be visual and not a crash, although in some scenarios it did seem to trigger a crash.

I think this highlights the need for a much larger test suite.

Sidebar

It also made me question the insert(_:at:) functions because it's not really clear what the index is. Should it insert it at the index in the context of the sections and providers (as it does now), or just sections?

For example, with:

• Section A
• Section Provider
  • Section B
  • Section C
• Section D
• Section E

If I insert a section at index 3 should it look like:

• Section A
• Section Provider
  • Section B
  • Section C
• New Section
• Section D
• Section E

or

• Section A
• Section Provider
  • Section B
  • Section C
• Section D
• New Section
• Section E

The tests are currently essentially empty but at a minimum this makes sure things compile on Xcode 11.7 and 12.0.

@shaps80 I think this will need updates to the repo settings to enable GitHub Actions.

Introduced in #9. #12 adds GitHub actions to ensure this doesn't occur again.

Previously there was no way to get the index of a child or even know if a Section or SectionProvider is a child of a ComposedSectionProvider.

On top of these functions new insert(_:after:) and insert(_:before:) have been added for convenience.

The completion block is called when the animation has completed, but at the end of the updates block the data has already been updated (from the perspective of the UICollectionView).

Not calling reset() and setting defersUpdate = false could cause crashes e.g. due to changes being applied multiple times.

The best way to recreate this is to have a series of updates occur very close to each other but with the "Slow Animation" setting enabled in the simulator.

This change might also apply to table views, I have not checked.

The crash would often be along the lines of "Tried to delete section at index 11 but there were only 11 sections" or would occur in elementsProvider(for:) with the error "No UI configuration available for section 11" when performing a non-data change e.g. a reload of a section

Sometimes it's useful to have multiple different cell types within a single section, for example to use sticky headers or to reduce to need for many layers of composition (which can be tricky to get animation right for).

This isn't a breaking change for consumers.

I think some of the naming should be improved if this were to be merged.

Draft because I want feedback on the impact of the feature itself rather than specifically the code.

The package(s) being submitted are:

• HashableByKeyPath
• Persist

Checklist

I have either:

• [x] Run swift ./validate.swift.

Or, checked that:

• [ ] The package repositories are publicly accessible.
• [ ] The packages all contain a Package.swift file in the root folder.
• [ ] The packages are written in Swift 4.0 or later.
• [ ] The packages all contain at least one product (either library or executable).
• [ ] The packages all have at least one release tagged as a semantic version.
• [ ] The packages all output valid JSON from swift package dump-package with the latest Swift toolchain.
• [ ] The package URLs are all fully specified including https and the .git extension.
• [ ] The packages all compile without errors.

Xcode does not support ed25519 SSH keys so if a user uses ed25519 keys to authenticate with GitHub the connection will fail. HTTPS does not require authentication (for public repos)

Config from https://developer.apple.com/documentation/avfoundation/cameras_and_media_capture/avcam_building_a_camera_app was used to make it easier to people checking out the repo to run the sample code. Only the team needs to be changed with this config.