TeachingKidsProgramming/TeachingKidsProgramming.Source.Java

Separate tkp-student-full build into profiles

Closed this issue · 0 comments

Can we create 2 POM profiles?

  1. A profile that excludes deep dives, but compiles recipes.
  2. A profile that includes deep dives and excludes recipes.

The problem is that these things interfere with each-other. Since the deep dives look like tests, maven will interpret incomplete deep dives as test failures, and abort. Then codenvy will see that the build aborted, and not start the runner. This means that a student who completes SimpleSquare will never see the square if they have not completed all the Deep Dives first. So, deep dives must be excluded while working on recipes.

Conversely, a student who completes Simple Square, and moves on to the Deep Dive, will not see the Deep Dive run if Simple Square has a syntax error. Why should there be a syntax error when we assume the student finished Simple Square? Maybe they broke it during variations. If this happens then Maven will notice compilation errors during the build phase, and abort the build before reaching the test phase.

So, keeping these things totally separate would be desirable. POM profiles should be able to handle it, but we would need to see Codenvy's UI for selecting a non-default profile.