Skip to content

Re-export all of gcore in gstd and rename 'Image' node to 'Image Value' #2733

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 2 commits into from
Jun 19, 2025

Conversation

Firestar99
Copy link
Collaborator

@Firestar99 Firestar99 commented Jun 18, 2025

Requires #2723

code changes, for bulk renames see #2729

This PR and #2729 makes editor to only depend on gstd and removes it's dependency on gcore. Instead, all of gcore is reexported in gstd. This allows me to rip apart gcore without editor being affected, as long as I continue to reexport all nodes in gstd. Saving us a lot of conflicts down the line.

@Firestar99 Firestar99 changed the title Reexport all of gcore in gstd, Rename image node to image_value Reexport all of gcore in gstd, rename image node to image_value Jun 18, 2025
@Firestar99 Firestar99 marked this pull request as ready for review June 18, 2025 13:42
@Firestar99 Firestar99 merged commit fd0d480 into master Jun 19, 2025
5 checks passed
@Firestar99 Firestar99 deleted the rename_image_node branch June 19, 2025 20:58
@Keavon Keavon changed the title Reexport all of gcore in gstd, rename image node to image_value Re-export all of gcore in gstd, rename 'Image' node to 'Image Value' Jun 20, 2025
@Keavon Keavon changed the title Re-export all of gcore in gstd, rename 'Image' node to 'Image Value' Re-export all of gcore in gstd and rename 'Image' node to 'Image Value' Jun 20, 2025
Keavon pushed a commit that referenced this pull request Jun 20, 2025
…Value' (#2733)

* rename `image` node to `image_value`

* reexport the entirety of `gcore` in `gstd`
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