Github user trixpan commented on the issue:
https://github.com/apache/nifi/pull/1608
@pvillard31 thanks for the rebase. LGTM. Merged
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this
Github user pvillard31 commented on the issue:
https://github.com/apache/nifi/pull/1608
No problem at all @trixpan, I rebased against master.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have
Github user trixpan commented on the issue:
https://github.com/apache/nifi/pull/1608
@pvillard31 sorry for this but it seems PR is now in conflict with master.
Would you mind rebasing?
Cheers
---
If your project is set up for it, you can reply to this email and have your
reply
Github user pvillard31 commented on the issue:
https://github.com/apache/nifi/pull/1608
Just pushed a commit to address your comment let me know if you want me to
squash the PR.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as
Github user pvillard31 commented on the issue:
https://github.com/apache/nifi/pull/1608
Sorry for the late answer @trixpan, just coming back from holidays. You're
right, I should update the description, will do it later today. Thanks!
---
If your project is set up for it, you can
Github user trixpan commented on the issue:
https://github.com/apache/nifi/pull/1608
Shouldn't `static final AllowableValue MODE_MULTIFILE` ... be updated to
reflect new behavior?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub