Ability to return meta information when aborting from the 'before' hook about papaparse

[mholt]

I'll definitely consider this for version 3.0 which is well under way, because I think it could be really useful. In the mean time, how about using a variable defined in a scope that both hooks can access?

[mholt]

@achambers I'm implementing this feature tonight. I'm thinking that the before hook will work a little differently than it does now. Right now, you can return a variety of things: an object to redefine the config, "skip" to skip, false to abort, and nothing (undefined) to continue. I'm going to standardize it so that if you do return anything, you always return an object that can look like this:

{
    action: "abort",
    reason: "some reason",
    config: { /* custom config for just this file */ }
}

Each property is optional. Not returning anything or not specifying an action or specifying an action of "continue" will carry on as normal. The action can also be "abort" or "skip". Only an action of "abort" will raise an error, but the reason will be carried into the error handler (which I'm still figuring out). If you want to adjust the config for a certain file, you can specify it in the config property.

Sound good?

[mholt]

Closing unless this still needs attention. Thanks again for the feedback!

[achambers]

Sorry @mholt. Didn't get around to replying. Yep, that sounds like a reasonable solution. I'll work it in to what I have when you're done and will see how it feels.
Nice work mate.