-
Notifications
You must be signed in to change notification settings - Fork 1k
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
chore: enable test multi-python version #143
base: main
Are you sure you want to change the base?
chore: enable test multi-python version #143
Conversation
9d22588
to
480ffc8
Compare
Looks good. I noticed this removes caching. I'm not sure caching was ever working as I wanted in the CI, but I wonder if you could comment on this? |
They're basically the same thing, but with a simpler setup |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
https://github.com/orgs/community/discussions/26698#discussioncomment-3252954 can you try this |
https://github.com/microsoft/markitdown/actions/runs/12411289890 Maybe because this is not being triggered. |
Ok, let me try that, and some other things. In meetings this AM, but I'll get to this ASAP |
37c0ff9
to
2acc826
Compare
Update the GitHub Actions checkout and setup-python versions to v4 and v5 respectively. Add Python 3.13 to the testing matrix in both the workflow and pyproject.toml to ensure compatibility with the latest Python release.
2acc826
to
fde036f
Compare
Current Behavior:
The test suite currently runs only on a single Python version.
Proposed Changes:
Evidence
Before
After