Skip to content

Rename isExists to exists. Fixes #448 - #457

Merged
aemaeth-me merged 1 commit into
SwiftyJSON:masterfrom
CraigSiemens:master
Mar 15, 2016
Merged

Rename isExists to exists. Fixes #448#457
aemaeth-me merged 1 commit into
SwiftyJSON:masterfrom
CraigSiemens:master

Conversation

@CraigSiemens

Copy link
Copy Markdown
Contributor

isExists() isn't the correct way to phrase that statement and makes the code harder to read. Simply using exists() makes it easer to read and grammatically correct.

json["user"].exists()

@bcapps

bcapps commented Feb 22, 2016

Copy link
Copy Markdown
Contributor

👍 Really nice, simple change that improves the readability of the API.

@aemaeth-me

Copy link
Copy Markdown
Member

Nice!!!

aemaeth-me added a commit that referenced this pull request Mar 15, 2016
Rename isExists to exists. Fixes #448
@aemaeth-me
aemaeth-me merged commit 653cf8c into SwiftyJSON:master Mar 15, 2016
zetavg added a commit to zetavg/react-native-ios-charts that referenced this pull request May 2, 2016
isExists() for JSON has not been add upon SwiftyJSON 2.3.1, and is renamed to exists() in this later PR: SwiftyJSON/SwiftyJSON#457.

Specifying "~> 2.3.0" might not work, and let users run into the following error:

```
error: value of type 'JSON' has no member 'isExists'
```
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