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

Added an option to stop at splitter #9

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

mikkelking
Copy link

Thanks for writing the medium article and contributing this package, it has saved me some considerable work.

In the test data I assembled, I used emails from our system, and discovered that the some links in the code are wrapped by the email system. and planer detects them as normal lines, and therefore includes the whole email instead of just the reply text.

I made a change to add an option to ignore these false positives and only take the response up to the splitter.

I also introduced some debug statements, thanks to the excellent npm debug module https://github.com/visionmedia/debug. To activate debugging output, prefix your command with DEBUG=planer

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