-
Notifications
You must be signed in to change notification settings - Fork 175
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
Add tests for min dependencies and update test matrix for min and max server versions #183
Conversation
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.
A comment on hardcoded versions.
.github/workflows/test-python.yml
Outdated
- name: Start MongoDB on MacOS | ||
if: ${{ startsWith(runner.os, 'macOS') }} | ||
run: | | ||
brew tap mongodb/brew | ||
brew install mongodb/brew/mongodb-community@${MONGODB_VERSION} | ||
brew services start mongodb-community@${MONGODB_VERSION} | ||
brew install mongodb/brew/[email protected] |
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.
How come this change?
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.
This is the highest version supported on brew right now, but not the highest available with the GitHub action on Ubuntu.
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.
Actually, I must have had it misconfigured before, it works with 8.0.
.github/workflows/test-python.yml
Outdated
uses: astral-sh/setup-uv@v5 | ||
with: | ||
enable-cache: true | ||
python-version: "3.9" |
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.
Similar to comment about ${MONGODB_VERSION} => 7.0,
MIN_PYTHON_VERSION: "3.9", MONGODB_VERSION: "4.0" seem like a good idea. What am I missing?
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.
Updated
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.
The latest env changes it make it much easier to see what you're after. Thanks!
Run tests against minimum python packages and supported MongoDB version. Also test against the latest MongoDB version.