Skip to content
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

[workspace] Upgrade vtk_internal to latest commit #22597

Closed

Conversation

BetsyMcPhail
Copy link
Contributor

@BetsyMcPhail BetsyMcPhail commented Feb 6, 2025

Remove VTK patches that have been upstreamed.

Replaces #22565

Towards: #22536


This change is Reviewable

Remove VTK patches that have been upstreamed.
@BetsyMcPhail BetsyMcPhail added the release notes: none This pull request should not be mentioned in the release notes label Feb 6, 2025
@BetsyMcPhail BetsyMcPhail added the status: single reviewer ok https://drake.mit.edu/reviewable.html label Feb 6, 2025
@BetsyMcPhail BetsyMcPhail added release notes: fix This pull request contains fixes (no new features) and removed status: single reviewer ok https://drake.mit.edu/reviewable.html release notes: none This pull request should not be mentioned in the release notes labels Feb 6, 2025
@jwnimmer-tri
Copy link
Collaborator

BTW When ready, we'll have @SeanCurtis-TRI do both reviews.

Copy link
Contributor

@rpoyner-tri rpoyner-tri left a comment

Choose a reason for hiding this comment

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

+@SeanCurtis-TRI for both reviews. Just getting this out of the "no one is assigned" list.

Reviewable status: LGTM missing from assignee SeanCurtis-TRI(platform), needs at least two assigned reviewers

@jwnimmer-tri
Copy link
Collaborator

This is not ready for review +(status: do not review) -@SeanCurtis-TRI.

Copy link
Contributor Author

@BetsyMcPhail BetsyMcPhail left a comment

Choose a reason for hiding this comment

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

@sankhesh will investigate when he returns from vacation, the week of 2/25

Reviewable status: LGTM missing from assignee sankhesh, needs platform reviewer assigned, needs at least two assigned reviewers

@jwnimmer-tri
Copy link
Collaborator

Superseded by #22702.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
release notes: fix This pull request contains fixes (no new features) status: do not review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants