skip_rows for Excel_Workbook.read.

[jdunkerley]

Pull Request Description

• Adds skip_rows to Excel_Workbook.read.
• Support for read by index.

Checklist

Please ensure that the following checklist has been satisfied before submitting the PR:

• The documentation has been updated, if necessary.
• Screenshots/screencasts have been attached, if there are any visual changes. For interactive or animated visual changes, a screencast is preferred.
• All code follows the
  Scala,
  Java,
  TypeScript,
  and
  Rust
  style guides. In case you are using a language not listed above, follow the Rust style guide.
• Unit tests have been written where possible.
• If meaningful changes were made to logic or tests affecting Enso Cloud integration in the libraries,
  or the Snowflake database integration, a run of the Extra Tests has been scheduled.
  • If applicable, it is suggested to paste a link to a successful run of the Extra Tests.

[AdRiley]

Looks good. Be good to have a few tests that include headers too to show how that works with skip and limit. I guess skip is the interesting one.

[jdunkerley]

Looks good. Be good to have a few tests that include headers too to show how that works with skip and limit. I guess skip is the interesting one.

Will be working on headers next - its an interesting question... should you read header then skip rows or vice versa!
Currently we do the later

[AdRiley]

Looks good. Be good to have a few tests that include headers too to show how that works with skip and limit. I guess skip is the interesting one.

Will be working on headers next - its an interesting question... should you read header then skip rows or vice versa! Currently we do the later

Yeah. I think skip rows then read the header. Get rid of the rubbish people like to add before their tables in Excel.