markap14 commented on PR #112: URL: https://github.com/apache/nifi-api/pull/112#issuecomment-6366925182
[Claude Opus 5.5] Thanks for the review @exceptionfactory. I pushed 23ba1d9 to address the feedback: removed the `public` modifiers from `TestStateManager`, moved the repeated literals in `TestStateManager` into static final constants, and changed `StateMap.toMap(Map)` to copy into a `LinkedHashMap` so the output order is consistent. I left the naming question for `toMap(Map)` alone, since @markap14 replied on that thread; I'll leave that one for the two of you to decide. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
