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

Issue661 #662

Closed
wants to merge 2 commits into from
Closed

Issue661 #662

wants to merge 2 commits into from

Conversation

gmweinberg
Copy link
Contributor

Change the python versions from tests to 3.10, 3.11, 3.12

@sh4nks
Copy link
Member

sh4nks commented Apr 10, 2024

Thanks! Not sure why GitHub Actions uses 3.1 instead of "3.10". Could you look into that? I guess putting the versions in quotes will fix it. Seems like there is still a error in the tests but I believe that this is a pytest issue which we could fix in a later PR.

@gmweinberg
Copy link
Contributor Author

It looks like you are correct, the versions should be strings, in the github actions they are using strings, and in hindsight it makes sense that without quotes it interprets 3.10 as being the same as 3.1.

Some of the other problems seem to be caused by invalid dates. The tests fail when you feed it a date like
birthday="25 06 2000"
but succeed when you give it birthday="2000-06-25". I'll look into it a little bit more.

@gmweinberg
Copy link
Contributor Author

I'll close this pull request as i have an updated one.

@gmweinberg gmweinberg closed this Apr 11, 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.

None yet

2 participants