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

Remove unused Rx bits #670

Conversation

allenhumphreys
Copy link
Collaborator

I'm basing the removal of RxSwift on your branch, and here are some small pieces of house keeping that are safe to take into the base branch now! (Also, master won't build for me because of some package nonsense)

@allenhumphreys allenhumphreys requested a review from insidegui May 28, 2023 18:30
@allenhumphreys allenhumphreys force-pushed the ah/rx-removal-house-keeping-rambo-base branch from 2374a94 to f93ab8d Compare May 28, 2023 18:34
@allenhumphreys
Copy link
Collaborator Author

allenhumphreys commented May 29, 2023

@bcmn thanks for the approve! Wasn't sure if you were still watching. I reformulated this cleanup in #672 against master and included the SPM fixes from Gui's branch so master will be buildable sooner than later.

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