-
Notifications
You must be signed in to change notification settings - Fork 16
Add replace(), __getitem__, and __iter__ to pipeline #157
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
base: master
Are you sure you want to change the base?
Conversation
7fa936b
to
5ddde13
Compare
Thanks again. Could you add entries in the Changelog under "Unreleased"? The same applies to the other PRs. Happy to merge when the pipelines pass. |
Ah! I forgot to run tox! Fixed now. |
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #157 +/- ##
==========================================
- Coverage 96.02% 95.95% -0.07%
==========================================
Files 48 48
Lines 3171 3217 +46
==========================================
+ Hits 3045 3087 +42
- Misses 126 130 +4 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
14f9e58
to
66ad327
Compare
Should be good now - it made sense to add |
This allows one to more easily find registered functions (such as the broken language trimmers) in a pipeline and replace them (with less broken ones). Fixes #155