emc-mongoose/scenario-converter-3to4

Issues

• bug with extra quotes

  #37 opened 6 years ago by veronikaKochugova
  0

• Delete empty config in PipelineLoad

  #33 opened 6 years ago by veronikaKochugova
  0

• Fix the example scenarios

  #34 opened 6 years ago by veronikaKochugova
  0

• Unused & incorrect variables must be deleted

  #32 opened 6 years ago by veronikaKochugova
  0

• Code style: the condition looks terrifying. Please don't do so

  #30 opened 6 years ago by akurilov
  0

• "new ArrayList" inside the string wouldn't work

  #29 opened 6 years ago by akurilov
  0

• constant should be final

  #31 opened 6 years ago by akurilov
  0

• converter.iml file is not needed, please remove it

  #28 opened 6 years ago by akurilov
  0

• Add a CLI option to output the scenario to the file with the same name but different extension

  #26 opened 6 years ago by akurilov
  0

• README.md is missing

  #14 opened 6 years ago by akurilov
  1

• PipelineLoad

  #24 opened 6 years ago by veronikaKochugova
  0

• MixedLoad

  #25 opened 6 years ago by veronikaKochugova
  0

• Reduce the cohesion

  #17 opened 6 years ago by akurilov
  1

• Bug with escaping quotes in ProcessBuilder()

  #23 opened 6 years ago by veronikaKochugova
  1

• Parameter --run-scenario

  #5 opened 6 years ago by veronikaKochugova
  0

• count/time limits are purged by the concurrency limit: should be merged instead

  #22 opened 6 years ago by akurilov
  0

• MONGOOSE_DIR is env var -> won't work inside the string

  #21 opened 6 years ago by akurilov
  0

• Asynchronous execution

  #20 opened 6 years ago by akurilov
  0

• Use the args only, plus ability to handle multiple args as file list

  #15 opened 6 years ago by akurilov
  0

• Meaningless interface implementation

  #19 opened 6 years ago by akurilov
  0

• Missorted modifiers "final private" -> "private final"

  #16 opened 6 years ago by akurilov
  0

• Still not enough finals...

  #18 opened 6 years ago by akurilov
  0

• Always try to make all non-primitive fields to be final

  #11 opened 6 years ago by akurilov
  0

• final, final, final...

  #12 opened 6 years ago by akurilov
  0

• Having static but not constant field seems to be very dangerous practice

  #8 opened 6 years ago by akurilov
  0

• Parallel Step

  #7 opened 6 years ago by veronikaKochugova
  0

• Rename: Converter -> ConverterImpl, IConverter -> Converter

  #10 opened 6 years ago by akurilov
  0

• Please don't use "I" prefix for the interfaces

  #9 opened 6 years ago by akurilov
  0

• 4 types of "for" loop

  #6 opened 6 years ago by veronikaKochugova
  0

• Let's use CLI args as a scenario paths instead of the code above

  #1 opened 6 years ago by akurilov
  1

• "final" is neccessary for 1) local vars if possible 2) method args 3) class fields if possible

  #3 opened 6 years ago by akurilov
  0

• "JSON" -> "Json" according the modern guidelines

  #4 opened 6 years ago by akurilov
  0

• "final" is unneccessary inside the interface

  #2 opened 6 years ago by akurilov
  0