Skip to content

Split operator.py into multiple files #420

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 6 commits into from
Mar 31, 2023
Merged

Split operator.py into multiple files #420

merged 6 commits into from
Mar 31, 2023

Conversation

eriknw
Copy link
Member

@eriknw eriknw commented Mar 29, 2023

This addresses some of the feedback from #410.

I've tried to do this in the past, but always failed. I think we've finally untangled things enough to allow this, b/c I succeeded this time!

@eriknw eriknw added the hygiene Improve code quality and reduce maintenance overhead label Mar 29, 2023
@coveralls
Copy link

coveralls commented Mar 29, 2023 •
edited
Loading

Coverage Status

Coverage: 99.471% (-0.02%) from 99.491% when pulling 81ea188 on eriknw:split_operator into 0139a83 on python-graphblas:main.

@eriknw
Copy link
Member Author

eriknw commented Mar 31, 2023

Thanks for reviewing Jim--merging!

@eriknw eriknw merged commit 40e548d into python-graphblas:main Mar 31, 2023
@eriknw eriknw mentioned this pull request May 8, 2023
Closed
24 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jim22k jim22k jim22k approved these changes

@SultanOrazbayev SultanOrazbayev Awaiting requested review from SultanOrazbayev

Assignees
No one assigned
Labels
hygiene Improve code quality and reduce maintenance overhead
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@eriknw @coveralls @jim22k