[ENH] Add tidyxl's xlsx_table to the io module.

[samukweku]

PR Description

Please describe the changes proposed in the pull request:

• Create DataFrame or dictionary of DataFrames from table(s) in xlsx sheet.

This PR resolves #992 .

PR Checklist

Please ensure that you have done the following:

1. PR in from a fork off your branch. Do not PR from <your_username>:dev, but rather from <your_username>:<feature-branch_name>.

2. If you're not on the contributors list, add yourself to AUTHORS.rst.

3. Add a line to CHANGELOG.md under the latest version header (i.e. the one that is "on deck") describing the contribution.
   • Do use some discretion here; if there are multiple PRs that are related, keep them in a single line.

Automatic checks

There will be automatic checks run on the PR. These include:

• Building a preview of the docs on Netlify
• Automatically linting the code
• Making sure the code is documented
• Making sure that all tests are passed
• Making sure that code coverage doesn't go down.

Relevant Reviewers

Please tag maintainers to review.

• @ericmjl

[codecov]

Codecov Report

Merging #990 (a6672db) into dev (ba46402) will increase coverage by 0.04%.
The diff coverage is 94.11%.

@@            Coverage Diff             @@
##              dev     #990      +/-   ##
==========================================
+ Coverage   96.28%   96.32%   +0.04%     
==========================================
  Files          77       77              
  Lines        2826     2859      +33     
==========================================
+ Hits         2721     2754      +33     
  Misses        105      105              

[ericmjl]

Great start, @samukweku! I put in some early review comments here. There are tests that are missing too, but I trust you're already working on them. Looking forward to reviewing!

[ericmjl]

I'm inclined to ignore the decrease in project test coverage; they're irrelevant to the PR. 100% of the patch diff being hit is good enough.

[ericmjl]

Btw, @samukweku, I think going forth you can use a branch off the pyjanitor repo, rather than a personal fork. Or otherwise, you can delete the old forks that you have and re-fork. (This is what I do with projects I'm not a maintainer on.) It helps eliminate the need to maintain a fork and keep the dev branch up-to-date.

[ericmjl]

I love the tests provided here, @samukweku, they're really comprehensive!

I have only a few nit-picky requests for changes. They are explicit requests for changes, rather than mere suggestions, because I think changing them will help with maintenance clarity. Hope you don't mind them!

[thatlittleboy]

Sorry, immediately thought of another point I would like to make in the review right as I pressed "Submit" 😅

[ericmjl]

I'm approving. I looked on CodeCov and saw that the missed patch is on the conditional import. I don't think it's easy to test that in the current testing environment, so we should let it go.

I'm going to approve! @thatlittleboy since you also did a big review on this one, feel free to approve and merge!

[thatlittleboy]

@samukweku There's a merge conflict in CHANGELOG.md because of the previous PR going through. Could you address this, then I'll approve and merge? Thanks!