Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add isValidIndexPath to UITableViewExtensions #441

Merged
merged 3 commits into from
Apr 11, 2018

Conversation

setoelkahfi
Copy link
Member

@setoelkahfi setoelkahfi commented Apr 11, 2018

🚀

This PR will support my next PR about safely scroll table view to index path. Because it mentioned that one PR only for one extension, so I separate it.

Checklist

  • I checked the Contributing Guidelines before creating this request.
  • New extensions are written in Swift 4.
  • New extensions support iOS 8.0+ / tvOS 9.0+ / macOS 10.10+ / watchOS 2.0+.
  • I have added tests for new extensions, and they passed.
  • All extensions have a clear comments explaining their functionality, all parameters and return type in English.
  • All extensions are declared as public.
  • I have added a changelog entry describing my changes.

@SwifterSwiftBot
Copy link

SwifterSwiftBot commented Apr 11, 2018

1 Message
📖 Thank you for submitting a pull request to SwifterSwift. The team will review your submission as soon as possible.

Generated by 🚫 Danger

@codecov
Copy link

codecov bot commented Apr 11, 2018

Codecov Report

Merging #441 into master will increase coverage by <.01%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #441      +/-   ##
==========================================
+ Coverage   91.63%   91.63%   +<.01%     
==========================================
  Files          58       58              
  Lines        2677     2679       +2     
==========================================
+ Hits         2453     2455       +2     
  Misses        224      224
Flag Coverage Δ
#ios 91.63% <100%> (ø) ⬆️
#osx 91.63% <100%> (ø) ⬆️
#tvos 91.63% <100%> (ø) ⬆️
Impacted Files Coverage Δ
...urces/Extensions/UIKit/UITableViewExtensions.swift 95.08% <100%> (+0.16%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 686c617...9933d5b. Read the comment docs.

@omaralbeik omaralbeik self-requested a review April 11, 2018 06:16
Copy link
Member

@omaralbeik omaralbeik left a comment

Choose a reason for hiding this comment

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

Thank you for your contribution @setoelkahfi :)
Could you please add a changelog entry describing your new extension, more info here.

@setoelkahfi
Copy link
Member Author

Will do @omaralbeik

@@ -4,6 +4,8 @@ The changelog for **SwifterSwift**. Also see the [releases](https://github.com/S
> # Upcoming release
>
> ### Added
- **UITableViewExtentions**:
- Added `isValidIndexPath(_ indexPath:)` method to check whether given IndexPath is valid within UITableView. [#441](https://github.com/SwifterSwift/SwifterSwift/pull/441) by [setoelkahfi](https://github.com/setoelkahfi).
> ### Changed
- **UITableViewExtentions**:
Copy link
Member

Choose a reason for hiding this comment

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

@setoelkahfi, could you please add your changes here, we don't have to create a new UITableViewExtentions title if it already exists

Copy link
Member Author

@setoelkahfi setoelkahfi Apr 11, 2018

Choose a reason for hiding this comment

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

Oh, so it should be in the 'Changed' section of UITableViewExtentions, @omaralbeik ?

Copy link
Member

Choose a reason for hiding this comment

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

Oops, sorry my bad @setoelkahfi I didn't notice it was under changed 😅

@omaralbeik omaralbeik merged commit 8e6a8ac into SwifterSwift:master Apr 11, 2018
@SD10
Copy link
Member

SD10 commented Apr 11, 2018

Thank you for contributing to SwifterSwift! I've invited you to join the SwifterSwift GitHub organization - no pressure to accept! If you'd like more information on what that means, check out our contributing guidelines. Feel free to reach out if you have any questions! 😃

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.

4 participants