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

Fix Swift 5.7 warnings. #52

Open
wants to merge 4 commits into
base: develop
Choose a base branch
from
Open

Conversation

robo-fish
Copy link
Contributor

The following warning is emitted for a number of function declarations when compiling with Swift 5.7 (Xcode 14 Beta):
"Non-'@objc' instance method in extensions cannot be overridden; use 'public' instead"

Changing the access levels of the functions from open to public resolves the warnings.

"Non-'@objc' instance method in extensions cannot be overridden; use 'public' instead"
@jormungand2006
Copy link

Hi @dehesa,
Apple has already released up to Xcode 14.2, do you plan to merge this pull request with the latest release?

'Foundation' was not imported by this file.
@inc-ali
Copy link

inc-ali commented May 14, 2023

@dehesa please accept this pull request :)

The workaround fixes an internal Swift 5.10 compiler bug,
where an error is thrown when 'NSDecimalString' is encountered
in the source code.

As a side effect, Swift tools version is upgraded to 5.9,
and the deployment platforms are drastically narrowed down.
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.

3 participants