Skip to content

Support SwiftPM#15

Merged
RomanPodymov merged 1 commit into
PromiseKit:masterfrom
lstomberg:master
Sep 10, 2024
Merged

Support SwiftPM#15
RomanPodymov merged 1 commit into
PromiseKit:masterfrom
lstomberg:master

Conversation

@lstomberg

Copy link
Copy Markdown
Contributor

Fixes #14

@lstomber

Copy link
Copy Markdown
Contributor

Any chance at getting this merged and a new tag created so that this project supports SPM?

@lstomberg lstomberg force-pushed the master branch 2 times, most recently from 6cdd8a9 to ca6a32a Compare May 15, 2020 05:49

#if !os(tvOS)

public struct AnimationOptions: OptionSet {

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Hello @lstomberg
Why do you need to re-define PMKAnimationOptions?

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Presumably for SwiftPM since it won't get the objective-c portions

@RomanPodymov

Copy link
Copy Markdown
Contributor

Hello @mxcl
Can you check this, please?

@mxcl

mxcl commented Sep 9, 2024

Copy link
Copy Markdown
Member

Looks fine

@RomanPodymov RomanPodymov merged commit 670a5b0 into PromiseKit:master Sep 10, 2024
@Vyazovoy

Vyazovoy commented Feb 7, 2025

Copy link
Copy Markdown

Hi, @RomanPodymov! Could you please make it work with the current PromiseKit version (currently it 6.8.3<7.0.0) and add a new version tag?

@RomanPodymov

Copy link
Copy Markdown
Contributor

Hello @Vyazovoy
Added v.4.0.0.

@Vyazovoy

Copy link
Copy Markdown

Thanks for so quick update, @RomanPodymov! 🙏

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.

Support Swift Package Manager

5 participants