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

Create python-package.yml #17

Closed
wants to merge 6 commits into from

Conversation

arteymix
Copy link
Member

@arteymix arteymix commented Apr 14, 2024

TODO

  • include staging and development servers in the build matrix (no need to run it on every Python version)

@arteymix
Copy link
Member Author

@oganm I fixed test_all.py, but I think test_tut.py needs to be rewritten. Do you have some time to do that?

@oganm
Copy link
Member

oganm commented Apr 15, 2024

Will have to replace both test_all.py and test_tut.py after the changes in the tables branch so I wouldn't worry about either for now. The updated tests are here here

@oganm
Copy link
Member

oganm commented Apr 16, 2024

As far as I can see i just need to change unittest with pytest?

@arteymix
Copy link
Member Author

pytest can run tests written with unittest.

In any case, I think using parametrized tests is good for testing all possible methods.

oganm added a commit that referenced this pull request May 7, 2024
@oganm
Copy link
Member

oganm commented May 8, 2024

Tests are re-implemented using the action from here

@oganm oganm closed this May 8, 2024
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