Skip to content

Conversation

@stevesoltys
Copy link

No description provided.

@TaeHagen TaeHagen self-requested a review April 14, 2025 14:16
Copy link
Collaborator

@TaeHagen TaeHagen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Mostly looks good, a few changes requested.

@stevesoltys stevesoltys requested a review from TaeHagen April 14, 2025 23:48
@TaeHagen
Copy link
Collaborator

Please note your contribution will be accepted under the MPL license, NOT the LGPL.

Upstream changed their license a while back, I never rebased for just that commit, but now it's here too.

remote-anisette = []
async = ["dep:async-trait"]
default = ["remote-anisette", "dep:remove-async-await", "async"]
default = ["remote-anisette-v3", "dep:remove-async-await", "async"]
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If I try to use clearadi, it does not compile, because remote-anisette-v3 is enabled. I cannot disable it, because icloud-auth always uses omnisette's default features. Either remove remote-anisette-v3 as a default feature, or make icloud-auth not use omnisette's default features.

@TaeHagen
Copy link
Collaborator

Thank you for your PR!

@TaeHagen TaeHagen merged commit d5fdf87 into OpenBubbles:master Apr 15, 2025
4 checks passed
@stevesoltys stevesoltys deleted the feature/ci-build branch April 15, 2025 02:14
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