Skip to content

Fix annotation renaming#109

Closed
d-o-g wants to merge 2 commits intoDiobf:masterfrom
d-o-g:patch-1
Closed

Fix annotation renaming#109
d-o-g wants to merge 2 commits intoDiobf:masterfrom
d-o-g:patch-1

Conversation

@d-o-g
Copy link

@d-o-g d-o-g commented Aug 6, 2025

No description provided.

EpicPlayerA10
EpicPlayerA10 requested changes Aug 6, 2025
View reviewed changes
Copy link
Collaborator

@EpicPlayerA10 EpicPlayerA10 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I would be very grateful if you would add tests to this case 🤗

@d-o-g
Copy link
Author

d-o-g commented Aug 31, 2025 •
edited
Loading

too much effort for something this small bro, i think this change is obvious enough anyway

@EpicPlayerA10
Copy link
Collaborator

EpicPlayerA10 commented Aug 31, 2025

I meant to just modify the existing test sample here and just add some simple annotation.

@d-o-g
Copy link
Author

d-o-g commented Aug 31, 2025

Done, it correctly renames type() as well as any references to it.

@EpicPlayerA10
Copy link
Collaborator

EpicPlayerA10 commented Aug 31, 2025 •
edited
Loading

Also you need to run mvn test for .dec files to be generated.

@d-o-g d-o-g closed this by deleting the head repository Sep 4, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@EpicPlayerA10 EpicPlayerA10 EpicPlayerA10 requested 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

@d-o-g @EpicPlayerA10