amirrajan / rubymotion-applied

RubyMotion documentation provided by the community. Submit a pull request to the docs for a free one year indie subscription.
Apache License 2.0
49 stars 10 forks source link

Standard Frameworks iOS template #101

Closed casper91 closed 6 years ago

casper91 commented 6 years ago

It seems like there are some standard frameworks that are loaded using ios-template

Perhaps this should be more explicit in the in the Rakefile of the project?

Maybe you don't want some of them (maybe they have to be there in order to work).

However while hacking I just kept thinking, why didn't I need to add those frameworks? (well they were defaults)

When do I use a frameworks not included in the defaults?

astechishin commented 6 years ago

What frameworks do you believe are unneeded by most apps?

I usually find the docs tell me when I need to add a framework (such as CoreData or CloudKit or AVKit)

casper91 commented 6 years ago

My concern was more, instead of the Rakefile containing:

# 5. If you need to reference any additional iOS libraries, use the config array below.
# ===========================================================================================
# app.frameworks << "StoreKit"

It could say something like:

# 5. If you need to reference any additional iOS libraries, use the config array below.
# ===========================================================================================
# Default's ['UIKit', 'Foundation', 'CoreGraphics', 'CoreFoundation', 'CFNetwork', 'CoreAudio']
# app.frameworks << "StoreKit"

Because I was using a lot of UIKit stuff and couldn't figure out why it worked without me adding it to the frameworks. Until I found them in the template.

amirrajan commented 6 years ago

@casper91 PR welcome. Just change this line: https://github.com/amirrajan/rubymotion-templates/blob/master/motion/project/template/ios/files/Rakefile.erb#L46

casper91 commented 6 years ago

Has been merged: https://github.com/amirrajan/rubymotion-templates/commit/f2396af90be0c56bfa3f1c91eb70af919703b74a