-
-
Notifications
You must be signed in to change notification settings - Fork 111
Added com.IvanMurzak.ReflectorNet #536
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
Conversation
IvanMurzak
commented
Jun 23, 2025
registry.json
Outdated
| "version": "0.16.4" | ||
| } | ||
| } | ||
| } |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Restore this and please open it with an IDE that uses .editorconfig to avoid this problemas
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sorry for that. Working on it.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixed
|
I see that your package depends on |
Hi @bdovaz , yes. It is a real dependency. Or better to say It was. I did the verification and found that the dependency were needed only for a minor operation in the project. I replaced that line of code and removed the dependency in the version 0.1.13. Original code:var dictInputParameters = inputParameters?.ToImmutableDictionary(
keySelector: p => p.name!,
elementSelector: p => reflector.Deserialize(p, logger)
);Updated code:var dictInputParameters = inputParameters?.ToDictionary(
keySelector: p => p.name!,
elementSelector: p => reflector.Deserialize(p, logger)
); |