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

Tests #14

Draft
wants to merge 30 commits into
base: main
Choose a base branch
from
Draft

Tests #14

wants to merge 30 commits into from

Conversation

nienna73
Copy link
Contributor

@nienna73 nienna73 commented Mar 29, 2023

What's in this PR:

The tests have been failing basically since the inception of this repo. This branch aims to fix that. The fun part: the tests don't work on my machine, but it's entirely possible they do work on the CD "machine", so I need to leave this PR here to trigger the tests while I work.

For now, the solution is to add # type: ignore to the end of every single import statement. Once that's done, other errors will reveal themselves. But for the time being, even though this isn't what should need to happen, this is what's happening. The other solution is to add consistent and diligent typing to the entire project, but considering I'm working on a time crunch, I chose the faster and less-good solution.

@nienna73 nienna73 marked this pull request as draft April 5, 2023 00:04
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.

1 participant