-
Notifications
You must be signed in to change notification settings - Fork 76
Added support for Carthage & Swift Package Manager. Project now supp… #16
base: master
Are you sure you want to change the base?
Conversation
…rts a universal framework called ProgressKit. Added Swift Playground to demo UI elements.
@dcilia Thank you for the pull request. Replacing demo app with playground was really a good idea. Also what version of Swift have you used here? |
|
||
|
||
static var allTests = [ | ||
("testExample", testExample), |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What does this do?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is an artifact of the template test class Xcode makes. This can be deleted.
@kaunteya are you using the |
|
@dcilia I am trying this on Xcode 8.3.3. I also checked the Swift version. It is Swift 3. It still shows
|
@dcilia updates? |
@kaunteya ---> try building the framework first; choose the destination as Mac or iOS Simulator and build. Then the playground should work. |
@kaunteya just checking in, were you able to get it to work? |
@dcilia I am trying it on Xcode 9. Frame work build successfully. But play ground was not working Let me know how I can fix this.. |
@dcilia Updates? |
@kaunteya I see from the screenshot you aren't running on the simulator , try building the framework first with the macOS in the destination scheme. Once you start adding progress UI for Are you on slack? email me directly if you want to pair. |
@dcilia Its working now. |
Hey @dcilia. Can you clarify the reason why the macOS demo app was replaced by Playground. The crux of ProgressKit is that you can create progress views with few lines of code. In this PR I see 3 problems
Let me know your views. |
@kaunteya That being said, I understand you want to highlight the fact that its simple to use in a nib / storyboard. My suggestion would be to instead create additional documentation in the form of a guide to highlight all the different great ways you can use this framework. This will allow potential and current users to obtain the relevant information you feel would be helpful. Your other questions:
Cheers 👍 |
@dcilia Mac app + Story board approach
Playground
Both methods have their ups and downs. So considering the use case of ProgressKit. Its reason of existence is that you can straight away drag a view and by just manipulating some IBOutlets you can have a smooth animated progress views. Dont you think we should demonstrate in way how it is expected to be used by people. I mean that is the whole purpose of having a demo app. So there would be 2 types of users.
Conclusion On top of that creating a separate documentation would be an overhead. Let me know your views.. |
All valid tradeoffs, as the creator of this project its your vision 👍. My understanding is that you have reservations around the use of the demo app / playground, which I leave to your decision whether to merge. (I would not be offended if you want to just close this pull request 👍 ). Cheers |
Any chance the Carthage and SPM support could be added without the other opinionated changes? |
I did not modify the
Podspec
. @kaunteya -> you can merge this PR in, and then modify and push an updatedPodspec
with a new a tag.Source files are at:
spec.source_files = 'Sources/*.*