Skip to content
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

Default to unsorted properties, add option to use sorted properties #888

Merged
merged 1 commit into from
Aug 30, 2021

Conversation

MikeEdgar
Copy link
Member

Fixes #87 . Makes the default object property order the same as the order from the class file, as read by Jandex 2.4.0. This also adds a new option to restore the previous sorted property order. Can not be merged without smallrye/smallrye-parent#270 and a new parent version.

@phillip-kruger
Copy link
Member

@MikeEdgar - 31 parent now released !

@MikeEdgar MikeEdgar changed the title Draft: Add option to use unsorted properties from Jandex Default to unsorted properties, add option to use sorted properties Aug 27, 2021
@MikeEdgar MikeEdgar marked this pull request as ready for review August 27, 2021 22:01
@MikeEdgar
Copy link
Member Author

@phillip-kruger , once this is merged, I'll update #870 as well.

@MikeEdgar MikeEdgar added this to the 2.1.11 milestone Aug 29, 2021
@phillip-kruger phillip-kruger merged commit fa3641b into smallrye:main Aug 30, 2021
@MikeEdgar MikeEdgar deleted the 87_unsorted_class_members branch September 19, 2021 11:06
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.

Don't automatically sort object fields
2 participants