pavankataria/SwiftDataTables

remove Pods from this repository

altagir opened this issue · 10 comments

altagir commented

I think you should remove the Example/Pods folder and Podfile.lock from this repository

  • clone this repo
  • cd Example (already a pods/ there with previous version)
  • pod update
    Installing SwiftDataTables 0.7.0 (was 0.6.31)

-> remove Pods from repo
-> remove Podfile.lock ? -> or at least update it with 0.7.0

cheers

pavankataria commented

I realise there's no value in adding the pods when it's just retrieving them from the source folder. It'll also make things lean and avoid an extra step of having to run pod install on the demo project. Plus it was redundant always lagging behind unless I updated after I uploaded the podspec which would cause more noisy commits.

I'll fix this in the next release. Do double check in case I leave anything out.

pavankataria commented

🎉 🎉 @altagir
This is now available as of version 0.7.1 🎉 🎉 https://github.com/pavankataria/SwiftDataTables/releases/tag/0.7.1

Let me know what you think 🎉

altagir commented

you need to clean current pods used in sample (Pods/ and Podfile as well as references from project)

cd SwiftDataTables/Example
pod deintegrate SwiftDataTables.xcodeproj
rm Podfile
pavankataria commented

It’s already been deintegrated along with a few other files, I used a plug-in from a cocoa pods creator; turns out doesn’t remove fake care of everything though deleting pod file etc is my responsibility. I’ll just push and change the tag commit reference

altagir commented

it wasn't deintegrated, was still present along with workspace in 0.7.1
I had to do above commands after a fresh pull from 0.7.1.
it will clean workspace and Pods
cheers

altagir commented

I put all 3 issues we were talking about in last 3 commits of
https://github.com/altagir/SwiftDataTables/commits/master

pavankataria commented

You're right, it wasn't deintegrated properly. I thought everything was dereferenced when I ran the deintegrate plugin. Turns out there were a few remaining pieces left as well as the configurations inside the project.

pavankataria commented

Please do try again and let me know if you still see the issue. I've tried on my side and it seems everything has been cleared.

  • 🎉1
pavankataria commented

I'll mark this issue as complete. Can reopen if there's any further issues.

altagir commented

sorry for late reply, been a bit overbusy :)
just verified, all perfect, thanks mate
regards

  • 👍1