-
Notifications
You must be signed in to change notification settings - Fork 238
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
Getting Reference to property 'transaction' in closure requires explicit use of 'self' to make capture semantics explicit in ImageManager #339
Comments
Seems old version compiler does not support I don't have any Xcode 14.x (Swift 5.7 ?), can you fix all the compile issue and submit PR for this ? Since I can not reproduce on my local Mac with Swift 5.10 or Swift 6 |
If you can compile by fixing this, please submit a PR about this change and I'll merge this. |
I have same problem and I can't solve it |
@softmastx Hi, can you just submit a PR for this ? It's just one line code. Since it's so simple I think it's suitable for other developers to contribute |
@dreampiggy I'm working on it. getting permission error and trying to fix. Could you please confirm that I have permission to push? |
You need to create a fork repo, then push to your fork repo's branch. And Create PR in https://github.com/SDWebImage/SDWebImageSwiftUI/pulls, choose your fork repo's branch as source branch, target branch is this repo's |
Closed by #340 |
Xcode 14.2
SDWebImageSwiftUI
The text was updated successfully, but these errors were encountered: