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

Make Manipulator an Enhancement and benefit from it #106

Merged
merged 4 commits into from
Nov 17, 2020

Conversation

exarkun
Copy link
Member

@exarkun exarkun commented Nov 11, 2020

Fixes #105

@coveralls
Copy link

coveralls commented Nov 11, 2020

Coverage Status

Coverage remained the same at 92.722% when pulling fcd829d on 105.manipulator-enhancement into ecf6368 on master.

src/imaginary/manipulation.py Outdated Show resolved Hide resolved
exarkun and others added 2 commits November 16, 2020 09:46
Perhaps should consider applying this elsewhere in Imaginary as well... Or perhaps coming up with a non-inheritance based mechanism for this.

Co-authored-by: Tristan Seligmann <mithrandi@mithrandi.net>
@mithrandi mithrandi merged commit 6d49062 into master Nov 17, 2020
@mithrandi mithrandi deleted the 105.manipulator-enhancement branch November 17, 2020 12:36
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.

Manipulator should be an Enhancement
3 participants