Skip to content

chore: Consolidate Duplicated Annotation Type Definitions#1900

Open
camielvs wants to merge 1 commit intomasterfrom
03-04-chore_consolidate_duplicated_annotation_type_definitions
Open

chore: Consolidate Duplicated Annotation Type Definitions#1900
camielvs wants to merge 1 commit intomasterfrom
03-04-chore_consolidate_duplicated_annotation_type_definitions

Conversation

@camielvs
Copy link
Collaborator

@camielvs camielvs commented Mar 5, 2026

Description

There is an Annotations type defined in utils/annotations and again in types/annotations leading to confusion and weird typecasting.

This PR consolidates it down into one Annotation type to match that of the Component Spec.

No change to app functionality.

Related Issue and Pull requests

Type of Change

  • Cleanup/Refactor

Checklist

  • I have tested this does not break current pipelines / runs functionality
  • I have tested the changes on staging

Screenshots (if applicable)

Test Instructions

  • Confirm annotations still work as expected.

Additional Comments

@github-actions
Copy link

github-actions bot commented Mar 5, 2026

🎩 To tophat this PR:

You can add the following URL parameter to your browser to tophat this PR:

`?tophat_location=03-04-chore_consolidate_duplicated_annotation_type_definitions/0287012`

Copy link
Collaborator Author

camielvs commented Mar 5, 2026

@camielvs camielvs force-pushed the 03-04-chore_consolidate_duplicated_annotation_type_definitions branch 3 times, most recently from 342cfe2 to c220f1a Compare March 5, 2026 02:27
@camielvs camielvs marked this pull request as ready for review March 5, 2026 02:28
@camielvs camielvs requested a review from a team as a code owner March 5, 2026 02:28
@camielvs camielvs changed the base branch from 03-04-feat_add_toggle_for_collapsing_nodes to graphite-base/1900 March 5, 2026 19:08
@camielvs camielvs force-pushed the 03-04-chore_consolidate_duplicated_annotation_type_definitions branch from c220f1a to 041ba7b Compare March 5, 2026 19:12
@camielvs camielvs force-pushed the graphite-base/1900 branch from 903d602 to 18933d0 Compare March 5, 2026 19:12
@graphite-app graphite-app bot changed the base branch from graphite-base/1900 to master March 5, 2026 19:13
@camielvs camielvs force-pushed the 03-04-chore_consolidate_duplicated_annotation_type_definitions branch 2 times, most recently from 97beba7 to 71ddb30 Compare March 5, 2026 22:31
@camielvs camielvs force-pushed the 03-04-chore_consolidate_duplicated_annotation_type_definitions branch from 71ddb30 to 0287012 Compare March 5, 2026 23:09
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.

1 participant