Skip to content

Detect added json items for static interfaces - #52

Merged
Zhengbo Li (zhengbli) merged 4 commits into
masterfrom
fixStaticInterface
Jan 25, 2016
Merged

Zhengbo Li (zhengbli) merged 4 commits into
masterfrom
fixStaticInterface

Conversation

@zhengbli

Copy link
Copy Markdown

The issue:
For interfaces marked as static in the spec, we used to not detect changes made in the external json files.

Related to PR #44

Comment thread Shared.fsx Outdated

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

why not let these be the point-free versions, eg

let getAddedItems = getItems addedItems

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good point!

@sandersn

Copy link
Copy Markdown
Member

Besides a minor comment, 👍

Zhengbo Li (zhengbli) pushed a commit that referenced this pull request Jan 25, 2016
Detect added json items for static interfaces
@zhengbli
Zhengbo Li (zhengbli) merged commit e93c0f9 into master Jan 25, 2016
@zhengbli
Zhengbo Li (zhengbli) deleted the fixStaticInterface branch January 25, 2016 18:53
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