Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Overcoat only requires PMKPromise, not all of PMK#85

Merged
sodastsai merged 1 commit into
Overcoat:masterfrom
mxcl:master
Jul 8, 2015
Merged

Overcoat only requires PMKPromise, not all of PMK#85
sodastsai merged 1 commit into
Overcoat:masterfrom
mxcl:master

Conversation

@mxcl

@mxcl mxcl commented Apr 28, 2015

Copy link
Copy Markdown
Contributor

Should you ever need more of PromiseKit it’s easy to add more subspecs.

Should you ever need more of PromiseKit it’s easy to add more subspecs.
@sodastsai sodastsai merged commit 9f54a55 into Overcoat:master Jul 8, 2015
@sodastsai

Copy link
Copy Markdown
Member

@mxcl thanks! 👍

@numerics

Copy link
Copy Markdown

When I was using Overcoat 2.1.1, cocoa pods installed PromiseKit 1.5.3... from your spec. Now it seems to be going backwards that Overcoat 3.0 uses PromiseKit 1.2...? Also, the Latest PromiseKit is 2.1.. is that suppose to be in Overcoat 3, (just transposed the numbers)?

@sodastsai

Copy link
Copy Markdown
Member

Hi @numerics, the version specified in the Podspec is ~> 1.2 which means you could install any version which is equal to or greater than 1.2 but smaller than 2.0. (>=1.2 and <2.0). So it's okay to keep using 1.5.3.

I don't really use PromiseKit, but I could still try to adopt the Overcoat to compatible with the newest PromiseKit. You're welcome to submit a PR for this too. 😄

@mxcl

mxcl commented Jul 17, 2015

Copy link
Copy Markdown
Contributor Author

I could submit a PR to upgrade to 2.x, it probably is minimal.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants