wynot12 commented on issue #69: [NEMO-2, 106, 117] Remove the /tests directory
URL: https://github.com/apache/incubator-nemo/pull/69#issuecomment-406014682
@wonook Thanks for review!
I've renamed `nemo-runtime-plangenerator` pacakge to `nemo-runtime-test`.
Please take a look :)
-
wynot12 commented on issue #69: [NEMO-2, 106, 117] Remove the /tests directory
URL: https://github.com/apache/incubator-nemo/pull/69#issuecomment-403849869
@wonook I've sent some commits that remove /runtime/test module with minor
relocation of tests.
But compiler-tests are still ind
wynot12 commented on issue #69: [NEMO-2, 106, 117] Remove the /tests directory
URL: https://github.com/apache/incubator-nemo/pull/69#issuecomment-403727665
I know that the package dependency of PR is not clear, but it's because of
the design of existing tests.
I believe this PR is better
wynot12 commented on issue #69: [NEMO-2, 106, 117] Remove the /tests directory
URL: https://github.com/apache/incubator-nemo/pull/69#issuecomment-403727192
I'll resolve conflicts soon.
This is an automated message from the Apa
wynot12 commented on issue #69: [NEMO-2, 106, 117] Remove the /tests directory
URL: https://github.com/apache/incubator-nemo/pull/69#issuecomment-403726944
@wonook Thanks for review and suggestion!
First, I didn't know `plangenerator` module is also for tests, since it's
name is conf
wynot12 commented on issue #69: [NEMO-2, 106, 117] Remove the /tests directory
URL: https://github.com/apache/incubator-nemo/pull/69#issuecomment-402928363
Here's the Nemo-internal dependency diagram after this PR.
https://docs.google.com/presentation/d/1S5EIFBDT7INfCTwRAW3ZcxrlGxVdDuvfT