search

blyscuit / Surveyer

0 stars 0 forks source link

Alamofire isn't in pod file, only AlamofireImage #5

Open edgarss opened 4 years ago

edgarss commented 4 years ago

Alamofire is installed as a dependancy of alamofireImage instead of being specified in podfile.

Why are you not using Alamofire5 yet?

blyscuit commented 4 years ago

Comparing upgrading from 4 to 5 and from 3 to 4, 5 has more mitigation needed. Many project in maintenance period is using AF4 and is on legacy code. It is reasonable that newer project is using AF5 but the downside is that everyone in the team will need to learn AF5 to be able to jump in at time of need. Personally I agree on using the latest major release but sometimes we also take in mind the resources (time & people) working or maintaining current projects.

I overlooked the specification Alamofire in podfile and accept the mistake.

edgarss commented 4 years ago

you dont have to upgrade to AF5 necessarily. we only use it in 2 projects as well. just thought to ask since you started a new project and AF 5.1 is the current stable version