Skip to content

Add Documentation regarding ArchRule priority#1671

Merged
hankem merged 1 commit into
mainfrom
feature/1633_improve_priority_documentation
Jul 24, 2026
Merged

Add Documentation regarding ArchRule priority#1671
hankem merged 1 commit into
mainfrom
feature/1633_improve_priority_documentation

Conversation

@TheManWhoStaresAtCode

Copy link
Copy Markdown
Member

As pointed out in #1633 we lack documentation how to configure the ArchRule priority.

Besides updating the userguide I also added JavaDoc for the priority enum. It might be beneficial to also add JavaDoc to the ArchRuleDefinition method priorities to support IDE users. However I think this should be a dedicated task where we document all public methods of that class or even all of the language package.

Comment thread archunit/src/main/java/com/tngtech/archunit/lang/Priority.java Outdated
Comment thread archunit/src/main/java/com/tngtech/archunit/lang/Priority.java Outdated
Comment thread archunit/src/main/java/com/tngtech/archunit/lang/Priority.java Outdated
@TheManWhoStaresAtCode

Copy link
Copy Markdown
Member Author

Thanks for the feedback @hankem. I updated the JavaDoc based on your suggestions.

Comment thread docs/userguide/007_The_Lang_API.adoc Outdated
@TheManWhoStaresAtCode
TheManWhoStaresAtCode force-pushed the feature/1633_improve_priority_documentation branch from 8eae0dd to 2f02151 Compare July 24, 2026 12:34
Comment thread docs/userguide/007_The_Lang_API.adoc Outdated
@TheManWhoStaresAtCode
TheManWhoStaresAtCode force-pushed the feature/1633_improve_priority_documentation branch from 2f02151 to bf6e08d Compare July 24, 2026 20:46
…Priority enum

Signed-off-by: Andreas Zöller <andreas.zoeller@tngtech.com>
@hankem
hankem force-pushed the feature/1633_improve_priority_documentation branch from bf6e08d to 10ee77f Compare July 24, 2026 21:05
@hankem
hankem merged commit 492b0e9 into main Jul 24, 2026
32 checks passed
@hankem
hankem deleted the feature/1633_improve_priority_documentation branch July 24, 2026 21:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants