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

Switch from nose to vanilla unittest (#78) #81

Merged
merged 1 commit into from
Jan 13, 2023

Conversation

zware
Copy link
Contributor

@zware zware commented Jan 2, 2023

Fixes #78

@juliangilbey
Copy link

Thanks @zware! I'll apply your patches in Debian until upstream applies them.

@juliangilbey
Copy link

Quick question, @zware - do your series of patches have the same licensing conditions as the original software? Thank!

@zware
Copy link
Contributor Author

zware commented Jan 5, 2023

I have given no special consideration to licensing, so I assume so :)

@anhaidgroup
Copy link
Owner

Hi Zachary and all, thanks so much for your contribution. In 2019 we created a startup building on these software packages. It was acquired in 2020 and I was away working full time in industry. I just came back and am rebuilding my research group. Hence there has been very little work on these in the past few years.

We will improve these packages, processing your pull requests, and more. This may take us a few weeks. But we will get it done and release a new version, as these packages have proven very useful (been used in many companies). Thank you for your patience, and if you have more to contribute, please do not hesitate to send it our way. Regards, AnHai Doan (UW-Madison).

@Anson-Doan Anson-Doan changed the base branch from master to bug_ci_testing January 13, 2023 22:07
@Anson-Doan Anson-Doan merged commit 2b67b60 into anhaidgroup:bug_ci_testing Jan 13, 2023
@zware zware deleted the drop_nose branch January 19, 2023 19:42
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.

Migrate tests away from nose
4 participants