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 skipEmptyLines and extendedRangeError #229

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

LiamKarlMitchell
Copy link

@LiamKarlMitchell LiamKarlMitchell commented Apr 7, 2023

Proposing a solution for #193
As well as a way to skip empty lines. #230

Or better yet could extract logic from writeRow to map cells to object and return that as the row?

This PR contains:

  • bugfix
  • feature
  • refactor
  • tests
  • documentation
  • metadata

Breaking Changes?

  • yes
  • no

If yes, please describe the breakage.

Please Describe Your Changes

Was wanting to be able to get more information about the line that has a RangeError, or really any error parsing.
Also a way to skip empty lines.

Proposing a solution for mafintosh#193
As well as a way to skip empty lines
Or better yet could extract logic from writeRow to map cells to object and return that as the row?
docs: skipemptylines
docs: extendedrangerror
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.

1 participant