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

Cocoapods spec #11

Closed
wants to merge 4 commits into from
Closed

Cocoapods spec #11

wants to merge 4 commits into from

Conversation

loicgriffie
Copy link

If you are ok with the cocoapods file you can just pusblish on cocoapods spec repos but before you do so, create a tag 1.0 of the repo

pod trunk push NAME.podspec

@chriswill0w
Copy link

@Appsido I think you have to make the convenience init of the UIAlertController public. Otherwise it will not compile.

@loicgriffie
Copy link
Author

loicgriffie commented Jan 26, 2018

@ltblueberry0 You may be right. I'll check that and more probably declare as open may be better than public

@chriswill0w
Copy link

@Appsido the method "addAction" and maybe some others need public/open, too.

@loicgriffie
Copy link
Author

loicgriffie commented Jan 29, 2018

@ltblueberry0 A lot of code change to make class + func public.

@dillidon Do you eventually plan to merge the request once work done ?

@dillidon dillidon closed this Jan 31, 2018
@afshin-hoseini
Copy link

afshin-hoseini commented Jan 31, 2018

@dillidon When do you plan to pull this?

@dillidon
Copy link
Owner

@afshin-hoseini sure, I've just pull some updates and going to do this task...

@loicgriffie
Copy link
Author

@dillidon seems you are not willing to merge my pull request. May I have your feedback why ?

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.

None yet

4 participants