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

Move examples out of the package and update readme #206

Merged
merged 3 commits into from
Aug 1, 2024

Conversation

clint-lawrence
Copy link
Collaborator

This came up somewhere in the threads discussion the VirtualMux related changes.

@clint-lawrence
Copy link
Collaborator Author

@daniel-montanari tagged you for require, because I think you suggested this? or at least we discussed it.

Copy link
Collaborator

@daniel-montanari daniel-montanari left a comment

Choose a reason for hiding this comment

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

Looks good, can we remove the examples folder from mypy.ini exclusions now please.
mypy.ini

I believe the rules on the example files at the bottom are also no longer required.

@clint-lawrence
Copy link
Collaborator Author

I believe the rules on the example files at the bottom are also no longer required.

Should be good now.

Copy link
Collaborator

@daniel-montanari daniel-montanari left a comment

Choose a reason for hiding this comment

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

Looks good

@clint-lawrence clint-lawrence merged commit ccac2c2 into master Aug 1, 2024
10 checks passed
@clint-lawrence clint-lawrence deleted the move-examples branch August 1, 2024 08:29
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