Skip to content

Conversation

@EgorkaKulikov
Copy link
Collaborator

Description

KryoHelper was located inside the instrumentation process.
I would like to use this serialization tool for communication with Spring analysis process too, so I moved it to the common project.

How to test

Manual tests

Verify that there is no regressions is serialization in instrumented process.

Self-check list

  • I've set the proper labels for my PR (at least, for category and component).
  • PR title and description are clear and intelligible.
  • I've added enough comments to my code, particularly in hard-to-understand areas.
  • The functionality I've repaired, changed or added is covered with automated tests.
  • Manual tests have been provided optionally.
  • The documentation for the functionality I've been working on is up-to-date.

@EgorkaKulikov EgorkaKulikov added ctg-enhancement New feature, improvement or change request comp-rd Issue is related RD labels May 11, 2023
@IlyaMuravjov IlyaMuravjov force-pushed the egor/move_kryo_helper branch from 26a7276 to 038601a Compare July 5, 2023 15:15
@IlyaMuravjov IlyaMuravjov merged commit 93be3c2 into main Jul 5, 2023
@IlyaMuravjov IlyaMuravjov deleted the egor/move_kryo_helper branch July 5, 2023 16:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

comp-rd Issue is related RD ctg-enhancement New feature, improvement or change request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants