Comments (4)
Great, all in one ! I can't wait to see these changes 🙂
from arch-unit-maven-plugin.
thanks for the suggestion, I think I see where you're going, and yes, it makes sense.
But one thing though, because I have a doubt on one use case : with your approach, all predefined rules have to be in PreconfiguredRules
in the "official" plugin distribution, right ? so people can't add their own private preconfigured rules in a jar and reference them in the config, right ?
from arch-unit-maven-plugin.
Yes, people would not be able to reference their custom rule as a preconfigured rule, but would have to use a configurable rule 😉.
For me this seems like a natural behavior though, preconfigured rules are delivered with the plugin, if I want to plug in a custom rule, I use the configurable rule API.
I mean, you could add a rule location to preconfigured rules as well, which defaults to PreconfiguredRules
, but then you kind of have configurable rule 😉
from arch-unit-maven-plugin.
yes, you're right, nobody would be totally blocked as such.
alright, let's see how this would go then ! We are very happy to get the inputs from various people, even if it means introducting a breaking change in the config. The project is still quite young, so hopefully not too many people are impacted ;-)
And if they are, maybe they can use CI-droid to replicate the changes in their many repositories !
from arch-unit-maven-plugin.
Related Issues (20)
- Classes under excluded paths are checked by configurableRules. HOT 2
- Configuration get's mixed up in multi module build HOT 7
- 2.6.x no longer works with aggregate POM HOT 3
- Update minimum build plugin core version in README HOT 2
- is there any plan to support jdk15? HOT 4
- @ArchTest of type ArchTests are not picked up by plugin HOT 4
- excludedPath generated-sources leads to StringIndexOutOfBoundsException: String index out of range: -1 HOT 1
- parallel execution of arch-test gives maven warning HOT 1
- can we mark this plugin @threadSafe HOT 3
- Is there a ImportOption donotincludeTests option? HOT 1
- Import classes from Main and Test scope
- Change log level from info to warning when noFailOnError=true HOT 2
- fix all maven warnings during build + make sure tests are executed HOT 1
- Execute rule on the aggregator ('pom' packaging) HOT 1
- Skip analysis for projects without source files HOT 6
- Plugin error when updating from 4.0.0 to 4.0.1 HOT 1
- Allow to exclude single checks from a configurable rule
- How share the rules between services
- Question: should my custom rules extend the class com.societegenerale.commons.plugin.rules.ArchRuleTest?
- Generate HTML Report
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from arch-unit-maven-plugin.