Skip to content

FIX: Text in prims#1992

Merged
svandenb-dev merged 20 commits intomainfrom
text_in_prims
Apr 7, 2026
Merged

FIX: Text in prims#1992
svandenb-dev merged 20 commits intomainfrom
text_in_prims

Conversation

@gkorompi
Copy link
Copy Markdown
Collaborator

@gkorompi gkorompi commented Apr 6, 2026

Adding primitivecollection and text types of primitives so customer can access them. @svandenb-dev

closes #1993

@gkorompi gkorompi changed the title Text in prims FIX: Text in prims Apr 6, 2026
Copy link
Copy Markdown
Collaborator

@svandenb-dev svandenb-dev left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Add a test please. Simple one with defining text and accessing attribute.

@codacy-production
Copy link
Copy Markdown

codacy-production bot commented Apr 6, 2026

Up to standards ✅

🟢 Issues 0 issues

Results:
0 new issues

View in Codacy

🟢 Metrics 14 complexity · 1 duplication

Metric Results
Complexity 14
Duplication 1

View in Codacy

TIP This summary will be updated as you push new changes. Give us feedback

@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Apr 6, 2026

Codecov Report

❌ Patch coverage is 64.28571% with 15 lines in your changes missing coverage. Please review.
✅ Project coverage is 69.99%. Comparing base (5cbae21) to head (8e861e3).
⚠️ Report is 71 commits behind head on main.

❌ Your patch status has failed because the patch coverage (64.28%) is below the target coverage (85.00%). You can increase the patch coverage or adjust the target coverage.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1992      +/-   ##
==========================================
- Coverage   70.20%   69.99%   -0.21%     
==========================================
  Files         261      265       +4     
  Lines       42416    43105     +689     
==========================================
+ Hits        29779    30173     +394     
- Misses      12637    12932     +295     
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@github-actions github-actions bot added the testing Anything related to testing label Apr 6, 2026
svandenb-dev
svandenb-dev previously approved these changes Apr 6, 2026
Copy link
Copy Markdown
Collaborator

@svandenb-dev svandenb-dev left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

svandenb-dev
svandenb-dev previously approved these changes Apr 6, 2026
Copy link
Copy Markdown
Collaborator

@svandenb-dev svandenb-dev left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@svandenb-dev svandenb-dev enabled auto-merge (squash) April 6, 2026 14:33
@gkorompi gkorompi requested a review from svandenb-dev April 6, 2026 16:19
Copy link
Copy Markdown
Collaborator

@svandenb-dev svandenb-dev left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@svandenb-dev svandenb-dev merged commit 2819b69 into main Apr 7, 2026
25 checks passed
@svandenb-dev svandenb-dev deleted the text_in_prims branch April 7, 2026 06:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

testing Anything related to testing

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Adding Primitive text in grpc filter_primitive, get_primitive_by_layer mishandle PrimitiveInstanceCollection (GDS import flow)

4 participants