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.

Sign up for GitHub

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

Jump to bottom

[Draft] JSON I/O to let pre-processors modify documents #125

Closed
wants to merge 11 commits into from
Closed

[Draft] JSON I/O to let pre-processors modify documents #125

wants to merge 11 commits into from

Conversation

mlange-42
Copy link
Owner

@mlange-42 mlange-42 commented Feb 20, 2021 •
edited
Loading

  • Move JSON-relevant modules to library crate
  • Add pre-processor section to config
  • run all pre-processors in the order given in config

Example pre-processors that use this implementation:

Fixes #53.

@mlange-42 mlange-42 force-pushed the feature/json-io branch 2 times, most recently from e4b3543 to cb09c34 Compare February 20, 2021 22:20
@mlange-42 mlange-42 added this to the Pre-processors milestone Feb 21, 2021
@mlange-42 mlange-42 mentioned this pull request Feb 21, 2021
Closed
@mlange-42 mlange-42 linked an issue Feb 21, 2021 that may be closed by this pull request
[Feature] Provide a possibility to hook in pre- or post-processors #53
Closed
@mlange-42 mlange-42 force-pushed the feature/json-io branch 3 times, most recently from 757649b to f2b41f9 Compare February 22, 2021 00:51
@mlange-42
use only original source files for hashing, not those generated by pe…
13f2ed3 
…r-processors; remove regex from lib dependencies; private Lock struct
@mlange-42 mlange-42 changed the title JSON I/O to let pre-processors modify documents [Draft] JSON I/O to let pre-processors modify documents Feb 22, 2021
@mlange-42 mlange-42 marked this pull request as draft February 22, 2021 11:12
@mlange-42
Copy link
Owner Author

Implemented more incrementally in #127, #128, #129 and #130

@mlange-42 mlange-42 closed this Feb 22, 2021
@mlange-42 mlange-42 deleted the feature/json-io branch February 22, 2021 17:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
Pre-processors
Development

Successfully merging this pull request may close these issues.

[Feature] Provide a possibility to hook in pre- or post-processors
1 participant
@mlange-42