-
Notifications
You must be signed in to change notification settings - Fork 7
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
Devcontainer and coverage reporting #176
Conversation
This is absolutely brilliant, thanks @dalmouiee! I will take a closer look at this a bit later, but I love that you've added these items. I also think running mypy as part of our GitHub pipelines is a great idea. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Awesome stuff @dalmouiee. Really appreciate your efforts here. Left a couple of comments/questions but I think we can merge this whenever you're ready.
Co-authored-by: Phil Chlap <[email protected]>
Hey @pchlap,
Just a few things I thought would be helpful for pydicer development moving forward, would love to hear your thoughts :)
Thanks!