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

Only skip first n lines in the first chunk and don't take the first line as header (#1045) #1046

Merged
merged 2 commits into from
Aug 5, 2024

Conversation

jkruke
Copy link
Contributor

@jkruke jkruke commented Mar 7, 2024

References #1045

tests/test-cases.js Show resolved Hide resolved
papaparse.js Outdated Show resolved Hide resolved
@foobarnes
Copy link

@jkruke @pokoli Is this ready to merge? Can I do any work on it to get it through? Pretty essentially on our end.

@jkruke
Copy link
Contributor Author

jkruke commented Apr 21, 2024

Actually, the reviewer @pokoli should re-review my actions regarding to his comments
From my point of view, no further work needs to be done then

@foobarnes
Copy link

@pokoli Any update here?

@jkruke
Copy link
Contributor Author

jkruke commented May 19, 2024

To get this done, maybe @foobarnes you wanna approve+merge this PR instead? :)

@foobarnes
Copy link

My bad, didn't realize I had permissions!

@jkruke jkruke requested a review from pokoli May 23, 2024 13:11
@jkruke
Copy link
Contributor Author

jkruke commented May 23, 2024

Hmm it's still not mergable because of unapproved changes, can we approve them somehow?

@newbeelearn
Copy link

can we merge this?

@jkruke
Copy link
Contributor Author

jkruke commented Jul 25, 2024

I'd love to merge this!

@jkruke
Copy link
Contributor Author

jkruke commented Jul 28, 2024

@pokoli (or @mholt): this PR cannot be merged without your action here

@mholt mholt merged commit cf4bded into mholt:master Aug 5, 2024
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.

6 participants