syky27 / LunchGuy

A simple application for retrieving lunch menus of selected restaurants around FIT CTU
http://obedar.fit.cvut.cz/
12 stars 5 forks source link

This repo is big as fekal #2

Closed hroncok closed 8 years ago

hroncok commented 8 years ago

And I consider that bad practice... :(

hroncok commented 8 years ago

Cloning into 'lunch_guy-ios'... remote: Counting objects: 623, done. remote: Compressing objects: 100% (4/4), done. remote: Total 623 (delta 0), reused 0 (delta 0), pack-reused 619 Receiving objects: 100% (623/623), 32.70 MiB | 630.00 KiB/s, done. Resolving deltas: 100% (173/173), done. Checking connectivity... done.

syky27 commented 8 years ago

There are used 2 huge frameworks

Nothing I can do about it.

hroncok commented 8 years ago

Is it necessary to keep it in the repo? Cannot you have some kind of podfile.lock that would tell what libraries to download and use?

hroncok commented 8 years ago

Oh wait, you already do and I guessed the name almost right.

syky27 commented 8 years ago

There are 2 options how manage this, you either have only Podfile, for installing dependencies and you basically don't push those dependencies into repo.

The other way is to push all necessary files for correct compilation to repo, because those third party libs might become unavailable in future.

I went with the second option for obvious reasons.

hroncok commented 8 years ago

OK, it's your repo. I still disagree but I respect your reasons.