Skip to content

New Field API #141

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

Merged
merged 16 commits into from
Nov 3, 2024
Merged

New Field API #141

merged 16 commits into from
Nov 3, 2024

Conversation

alexander-yevsyukov
Copy link
Collaborator

@alexander-yevsyukov alexander-yevsyukov commented Nov 1, 2024

This PR migrates Validation to new Field API recently introduced in ProtoData.

The migration required introduction of several extensions for FieldType. It is expected that they are going to be part of ProtoData sometime soon.

Build changes

  • Simplified arranging remote debug sessions on LaunchProtoData tasks.
  • Bumped dependencies on Logging, ToolBase, ProtoData, and Reflect.
  • config was also updated.

@alexander-yevsyukov alexander-yevsyukov self-assigned this Nov 1, 2024
@alexander-yevsyukov
Copy link
Collaborator Author

@armiol, "getting better every day": it finally builds. I hope to finish it tomorrow.

Copy link

codecov bot commented Nov 2, 2024

Codecov Report

Attention: Patch coverage is 0% with 64 lines in your changes missing coverage. Please review.

Please upload report for BASE (master@2aeeef6). Learn more about missing BASE report.
Report is 17 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff            @@
##             master     #141   +/-   ##
=========================================
  Coverage          ?   37.50%           
  Complexity        ?      362           
=========================================
  Files             ?      126           
  Lines             ?     2488           
  Branches          ?      209           
=========================================
  Hits              ?      933           
  Misses            ?     1493           
  Partials          ?       62           

@alexander-yevsyukov alexander-yevsyukov marked this pull request as ready for review November 3, 2024 14:08
@armiol armiol merged commit 7ebf5de into master Nov 3, 2024
7 checks passed
@armiol armiol deleted the new-field-api branch November 3, 2024 14:19
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