Skip to content

Reflection-free compatibility changes#214

Merged
MichaConrad merged 3 commits into
MichaCo:devfrom
RatherLogical:Compatibility-with-AOT-reflection-free-mode
Jun 30, 2024
Merged

Reflection-free compatibility changes#214
MichaConrad merged 3 commits into
MichaCo:devfrom
RatherLogical:Compatibility-with-AOT-reflection-free-mode

Conversation

@RatherLogical

Copy link
Copy Markdown
Contributor

APIs that use reflection will throw errors at runtime when publishing in AOT's reflection-free mode. I've changed the categoryName param in your logger to use a string instead of reflection. The only downside is that now you have to manually maintain the namespace and class names. This change allows your library to work for my use case, but if there is any other use of reflection in other parts of your lib we can fix that too.

@MichaConrad

Copy link
Copy Markdown
Collaborator

Don't really know why this is needed, feels kinda stupid, but ok why not

@MichaConrad MichaConrad merged commit 443d1bd into MichaCo:dev Jun 30, 2024
@RatherLogical RatherLogical deleted the Compatibility-with-AOT-reflection-free-mode branch June 30, 2024 22:08
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