[ 
https://issues.apache.org/jira/browse/TIKA-3879?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17617327#comment-17617327
 ] 

ASF GitHub Bot commented on TIKA-3879:
--------------------------------------

nddipiazza commented on code in PR #742:
URL: https://github.com/apache/tika/pull/742#discussion_r995166742


##########
tika-pipes/tika-emitters/tika-emitter-s3/src/main/java/org/apache/tika/pipes/emitter/s3/S3Emitter.java:
##########
@@ -16,33 +16,19 @@
  */
 package org.apache.tika.pipes.emitter.s3;
 
-import static org.apache.tika.config.TikaConfig.mustNotBeEmpty;

Review Comment:
   ah yeah. i'm getting fired up on a new laptop with new everything. so i'm 
setting up my checkstyle profile now





> add test containers test for s3 fetcher, emitter and pipe iterators
> -------------------------------------------------------------------
>
>                 Key: TIKA-3879
>                 URL: https://issues.apache.org/jira/browse/TIKA-3879
>             Project: Tika
>          Issue Type: Test
>          Components: tika-pipes
>            Reporter: Nicholas DiPiazza
>            Priority: Major
>
> need to add a testcontainers integration test for s3.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

Reply via email to