-
Notifications
You must be signed in to change notification settings - Fork 6
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 macOS #20
Remove macOS #20
Conversation
.DS_Store
Outdated
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this file should be included in the .gitignore
, preferably in your local global .gitignore
🙏🏼
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks great! Just please remove .DS_Store
from the root directory and from Sources
before merge 🙏🏼
Sources/.DS_Store
Outdated
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🗑️
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this should also be removed, no? maybe not if it's added by Xcode's Swift PM support, but maybe we can avoid loading this package in Xcode at all if it's unneeded?
We had a macOS target which is now obsolete since the removal of the CJNI target.
That means we can cleanup the whole codebase from any xcodeproject files and macOS-related things.