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

Unit tests for Root folder #328

Closed
wants to merge 4 commits into from
Closed

Conversation

tinmarr
Copy link
Member

@tinmarr tinmarr commented Aug 26, 2022

This includes unit tests for:

  • game.py
  • __init__.py
  • misc.py

MERGE AFTER #327

Hello reviewers!

Before approving this PR, please ensure the following:

  • Documentation is up to date
  • Relevent tutorials are up to date
  • Changelog is up to date
  • Test coverage is at 100%
  • All checks are passing
  • That the new code actually works

Once all this is done, you can be sure that the PR is ready to be merged. 🎉

This message has been added automatically. Please don't delete it.

@github-actions github-actions bot temporarily deployed to Preview- August 26, 2022 06:07 Inactive
@tinmarr tinmarr requested review from tsedan and mrElnekave and removed request for tsedan August 26, 2022 06:07
@tinmarr tinmarr mentioned this pull request Aug 26, 2022
5 tasks
@tinmarr tinmarr added the blocked This issue is dependent on the completion of another issue label Aug 26, 2022
@tinmarr tinmarr changed the base branch from main to pylance-typechecking-enabled August 26, 2022 06:22
@tinmarr tinmarr changed the title Unit tests progress Unit tests for Root folder Aug 26, 2022
@tinmarr tinmarr closed this Aug 26, 2022
@tinmarr tinmarr deleted the unit-tests branch August 26, 2022 06:26
@github-actions github-actions bot temporarily deployed to Preview- August 26, 2022 06:27 Inactive
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
blocked This issue is dependent on the completion of another issue
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant