Skip to content

Add annotations, so you don't have to use the ugly @Rule syntax thing#5

Merged
JanHolger merged 6 commits into
devfrom
feature/annotations
Dec 5, 2021
Merged

Add annotations, so you don't have to use the ugly @Rule syntax thing#5
JanHolger merged 6 commits into
devfrom
feature/annotations

Conversation

@x7airworker

@x7airworker x7airworker commented Dec 5, 2021

Copy link
Copy Markdown
Member 
@Rule("int(2, 3)")
int x;
// will now be
@IntegerRule(min = 2, max = 3)
int x;

I've wrote a little test and confirmed that the current state won't break.
image

@x7airworker x7airworker requested a review from JanHolger December 5, 2021 16:00
JanHolger
JanHolger requested changes Dec 5, 2021
View reviewed changes
Comment thread src/main/java/org/javawebstack/validator/Validator.java Outdated
Comment thread src/main/java/org/javawebstack/validator/Validator.java Outdated
Comment thread src/main/java/org/javawebstack/validator/Validator.java Outdated
@x7airworker x7airworker requested a review from JanHolger December 5, 2021 16:40
@JanHolger JanHolger merged commit e96fe03 into dev Dec 5, 2021
@x7airworker x7airworker deleted the feature/annotations branch December 5, 2021 17:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@JanHolger JanHolger JanHolger approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@x7airworker @JanHolger