Skip to content

IGListIndexSetResult and IGListIndexPathResult docs updates use old index#413

Closed
diogot wants to merge 1 commit into
Instagram:masterfrom
diogot:update-list-index-documentation
Closed

IGListIndexSetResult and IGListIndexPathResult docs updates use old index#413
diogot wants to merge 1 commit into
Instagram:masterfrom
diogot:update-list-index-documentation

Conversation

@diogot
Copy link
Copy Markdown
Contributor

@diogot diogot commented Jan 13, 2017

Changes in this pull request

Updated documentation to fix #408

Pull request checklist

  • All tests pass. Demo project builds and runs.
  • I added tests, an experiment, or detailed why my change isn't tested.
  • I added an entry to the CHANGELOG.md for any breaking changes, enhancements, or bug fixes.
  • I have reviewed the contributing guide

@diogot
Copy link
Copy Markdown
Contributor Author

diogot commented Jan 13, 2017

Is it necessary to add anything to CHANGELOG.md?

@coveralls
Copy link
Copy Markdown

coveralls commented Jan 13, 2017

Coverage Status

Coverage remained the same at 98.065% when pulling c07f0b8 on diogot:update-list-index-documentation into 279f348 on Instagram:master.

1 similar comment
@coveralls
Copy link
Copy Markdown

Coverage Status

Coverage remained the same at 98.065% when pulling c07f0b8 on diogot:update-list-index-documentation into 279f348 on Instagram:master.

@rnystrom
Copy link
Copy Markdown
Contributor

Thanks @diogot! No don't worry about the changelog 😄

@facebook-github-bot
Copy link
Copy Markdown
Contributor

@rnystrom has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator.

@diogot diogot deleted the update-list-index-documentation branch January 13, 2017 16:04
@jessesquires jessesquires added this to the 2.2.0 milestone Jan 13, 2017
jessesquires pushed a commit that referenced this pull request Jan 13, 2017
…ndex

Summary:
Updated documentation to fix #408

- [X] All tests pass. Demo project builds and runs.
- [X] I added tests, an experiment, or detailed why my change isn't tested.
- [ ] I added an entry to the `CHANGELOG.md` for any breaking changes, enhancements, or bug fixes.
- [X] I have reviewed the [contributing guide](https://github.com/Instagram/IGListKit/blob/master/.github/CONTRIBUTING.md)
Closes #413

Differential Revision: D4415035

Pulled By: rnystrom

fbshipit-source-id: 2c477c515c6f1346ce6db48d4ba865099cf197cd
@jessesquires jessesquires modified the milestones: 2.2.0, 3.0.0 Feb 23, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

IGListIndexSetResult and IGListIndexPathResult docs say updates use new index

5 participants