-
Notifications
You must be signed in to change notification settings - Fork 283
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
archunit-example should have (completely) self-contained build #30
Comments
But those 53 tests are tests only for |
By self-contained you mean, it's a completely separate project, where you add the Maven coordinates of
On the other hand, the only advantage would be to not have to compile the base |
as a potential new user, I don't care about these drawbacks - I need examples to get started. And: The example should be KISS (meaning: shortest possible setup time, just a single gradle build task ("run")). |
Okay, but at the moment, the process is
I wouldn't exactly call that a complicated setup 😉 (I would imagine most people would import the project into their IDE anyway, and run selected tests?) It would also be possible, to only run reasonable tasks, if I'm just trying to figure out, if there is no solution, that is convenient for an user of |
Peter, you're right, it's not really complicated. But as a user, I don't want the source + tests, just the example... You might move the examples to a distinct github repository and include them into the main ArchUnit via git. Then the examples might get their own build (+ doc), AND your goal of DRY is supported. The CI needs to pull changes from archunit-examples before performing the rest of the build,
does that sound reasonable? |
The submodules way has just the problem, that either it's not self contained (if I would throw the current |
Closed, as the examples have been extracted to a separate Github repository |
When running 'gradle test' on the archunit-example directory, I get:
53 tests, 0 failures, 53 ignored.
As an ArchUnit-user, I don't want to test the whole library, but only a (small) example...
(this issue belongs to the more general #22 )
The text was updated successfully, but these errors were encountered: