Skip to content

Use direct setter calls instead of using commons-beanutils#686

Closed
ppalaga wants to merge 1 commit intohighsource:masterfrom
ppalaga:i684.1
Closed

Use direct setter calls instead of using commons-beanutils#686
ppalaga wants to merge 1 commit intohighsource:masterfrom
ppalaga:i684.1

Conversation

@ppalaga
Copy link
Copy Markdown

@ppalaga ppalaga commented Mar 27, 2026

fix #684

This is an alternative approach to #685

@ppalaga
Copy link
Copy Markdown
Author

ppalaga commented Mar 27, 2026

I hope I have covered all properties which were supposed to be exposed. It looks like there is not much coverage of those features.

@laurentschoelens
Copy link
Copy Markdown
Collaborator

Missing properties, this should be reworked but could be interesting to remove commons-beanutils

@ppalaga
Copy link
Copy Markdown
Author

ppalaga commented Mar 27, 2026

Thanks for reviewing @laurentschoelens. Are you taking the effort over? Othewise I'd be ready to improve this PR if you give me some hits.

@laurentschoelens
Copy link
Copy Markdown
Collaborator

For the moment I'll go first with #685
You can run mvn build with all profile, which shows missing parameters for some project (but I dont think all)

In any case, that would go w/ 4.1

@ppalaga
Copy link
Copy Markdown
Author

ppalaga commented Mar 30, 2026

Thanks for the hint. Let me try with -Pall.

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.

Remove superfluous transitive dependency commons-collections:commons-collections

2 participants