[GitHub] flink issue #5820: [hotfix] [DataStream API] [Scala] removed unused scala im...

2018-05-28 Thread StefanRRichter 
Github user StefanRRichter commented on the issue:

https://github.com/apache/flink/pull/5820
  
This looks good to me 👍. Will merge.


---

[GitHub] flink issue #5820: [hotfix] [DataStream API] [Scala] removed unused scala im...

2018-04-12 Thread bowenli86 
Github user bowenli86 commented on the issue:

https://github.com/apache/flink/pull/5820
  
@zentol which scala checkstyle mvn plugin are you looking at? I didn't find 
any available ones, and I'm afraid there might will not be one.

scalastyle explicitly said that they will not support capturing unused 
imports at https://github.com/scalastyle/scalastyle/issues/193


---

[GitHub] flink issue #5820: [hotfix] [DataStream API] [Scala] removed unused scala im...

2018-04-12 Thread zentol 
Github user zentol commented on the issue:

https://github.com/apache/flink/pull/5820
  
I wouldn't bother cleaning up unused imports until we have a checkstyle 
rule in place to enforce it.


---