ENH: add integer sheetname support in read_excel

[cpcloud]

closes #4301

[jreback]

I guess a sheet names as '0' would have to be specified as as actual string to avoid confusion

[cpcloud]

yep. i should also add something to the docs acknowledging that and an example

[cpcloud]

i'll add a test for that to see how it's handled. not sure how xlrd deals with that

[jreback]

you could try to get a stringified first then fall back to positional if it is an integer
not sure if that is more confusing

[cpcloud]

so essentially prefer named sheets to indexing where possible

[jreback]

on 2nd thought I think passing an int try for index otherwise try named (like u have it)
no ambiguity

[cpcloud]

@jreback ot what is the Bounty label for?

[jreback]

Bounty label?

[cpcloud]

there's an issue label called "Bounty", just wondering what that's for...

[jreback]

no idea
maybe a standard label that git creates?
like bounty on stackoverflow

[jreback]

or it could be the quicker picker upper :)

[cpcloud]

😆

[ccverg]

@cpcloud @jreback it's the label for issues with bounties on Bountysource ;) (https://www.bountysource.com/trackers/50755-pydata-pandas)

[cpcloud]

@ccvergara oh cool didn't know about that thanks!

[cpcloud]

@jreback any last words?

[jreback]

I think u need to rebase as release notes
looks older

[cpcloud]

@jreback good 2 go?

[jreback]

this still looks like it has a conflict here?

[cpcloud]

oh sorry! didn't see that there was a repeat