Skip to content

Conversation

@sankhesh
Copy link
Collaborator

@sankhesh sankhesh commented Dec 7, 2021

Depends on Kitware/vtk-js#2179

@agirault agirault requested a review from jourdain December 7, 2021 21:25
@agirault
Copy link
Contributor

agirault commented Dec 7, 2021

Does react-vtk-js have some testing framework we could use here?

@jourdain
Copy link
Collaborator

jourdain commented Dec 7, 2021

unfortunately not. Just manual examples to test

Copy link
Collaborator

@jourdain jourdain left a comment

Choose a reason for hiding this comment

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

Seems reasonable. Glancing through the code seems fine.

@sankhesh
Copy link
Collaborator Author

@jourdain Could you please merge this one too?

@jourdain jourdain merged commit ceaff34 into Kitware:master Dec 16, 2021
@jourdain
Copy link
Collaborator

@sankhesh the pipeline is broken now...

@sankhesh
Copy link
Collaborator Author

@jourdain Thanks for letting me know. Fixing now.

@sankhesh sankhesh deleted the geometry2d branch December 16, 2021 21:27
@sankhesh
Copy link
Collaborator Author

Fixed in #49

@jourdain
Copy link
Collaborator

🎉 This PR is included in version 1.9.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants