Skip to content

Remove stale SpaceViewState struct from re_viewport#2352

Merged
Wumpf merged 1 commit intomainfrom
andreas/remove-old-spaceview-state
Jun 9, 2023
Merged

Remove stale SpaceViewState struct from re_viewport#2352
Wumpf merged 1 commit intomainfrom
andreas/remove-old-spaceview-state

Conversation

@Wumpf
Copy link
Copy Markdown
Member

@Wumpf Wumpf commented Jun 9, 2023

What

Followup to #2334: Remove old SpaceViewState struct from re_viewport now that all space view states have the same trait type

Checklist

PR Build Summary: https://build.rerun.io/pr/2352

Docs preview: https://rerun.io/preview/b468b7a/docs
Examples preview: https://rerun.io/preview/b468b7a/examples

@Wumpf Wumpf added 📺 re_viewer affects re_viewer itself 🚜 refactor Change the code, not the functionality labels Jun 9, 2023
Copy link
Copy Markdown
Contributor

@jleibs jleibs left a comment

Choose a reason for hiding this comment

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

Nice!

@Wumpf Wumpf merged commit c482cac into main Jun 9, 2023
@Wumpf Wumpf deleted the andreas/remove-old-spaceview-state branch June 9, 2023 14:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

📺 re_viewer affects re_viewer itself 🚜 refactor Change the code, not the functionality

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants