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

Reset to a clean slate and setup Swift #4

Merged
merged 11 commits into from
Dec 19, 2024
Merged

Reset to a clean slate and setup Swift #4

merged 11 commits into from
Dec 19, 2024

Conversation

bplattenburg
Copy link
Member

@bplattenburg bplattenburg commented Dec 13, 2024

This resets the general state of this repo to be used as an ongoing source of truth for the Presence Viewer and includes all of the required code & resources for it in Swift (copied from DittoSwiftTools).

There are a lot of changes in the diff, but it boils down to extracting everything from the DittoPresenceViewer target in DittoSwiftTools, making a few minor tweaks so it works from another package, and ripping out everything from the previous effort at building this several years ago.

As a first step, this very intentionally doesn't try to nail down the original non-minified source.

All shared resources can be found in the Shared folder, while everything for Swift specifically is under Swift. Unfortunately, until swift-tools 6.0 we need to copy them into the Swift section as well for Swift (see comment thread below).

To test, please use getditto/DittoSwiftTools#166.

Part of CXTOOLS-337 and CXTOOLS-358

@bplattenburg bplattenburg requested a review from a team December 13, 2024 18:35
@bplattenburg bplattenburg changed the title First pass at resetting this repo to a clean slate Reset to a clean slate and setup Swift Dec 13, 2024
Package.swift Outdated Show resolved Hide resolved
Copy link

linear bot commented Dec 13, 2024

@bplattenburg bplattenburg merged commit 1f1f912 into main Dec 19, 2024
@bplattenburg bplattenburg deleted the BP/reset branch December 19, 2024 14:27
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