Skip to content
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

Modified annotation #814

Merged
merged 3 commits into from
Feb 15, 2024
Merged

Modified annotation #814

merged 3 commits into from
Feb 15, 2024

Conversation

alexander-yevsyukov
Copy link
Contributor

This PR introduces the Modified annotation type which is needed for the cases when Spine tools update the code generated by other tools.

We need to tell who did the modifications because the changes ProtoData and Model Compilers apply are significant.

We also may want to modify the Protobuf generated code even further. So, relevant attributions are appropriate and wanted.

@alexander-yevsyukov alexander-yevsyukov self-assigned this Feb 15, 2024
@alexander-yevsyukov alexander-yevsyukov marked this pull request as ready for review February 15, 2024 00:27
Copy link

codecov bot commented Feb 15, 2024

Codecov Report

Merging #814 (71e833b) into master (8bb9617) will not change coverage.
The diff coverage is n/a.

Additional details and impacted files
@@            Coverage Diff            @@
##             master     #814   +/-   ##
=========================================
  Coverage     74.52%   74.52%           
  Complexity     1135     1135           
=========================================
  Files           181      181           
  Lines          4201     4201           
  Branches        335      335           
=========================================
  Hits           3131     3131           
  Misses          941      941           
  Partials        129      129           

@alexander-yevsyukov alexander-yevsyukov merged commit 79e2625 into master Feb 15, 2024
7 checks passed
@alexander-yevsyukov alexander-yevsyukov deleted the modified-annotation branch February 15, 2024 20:02
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