-
Notifications
You must be signed in to change notification settings - Fork 8
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
test #49
Conversation
This is a toy pr (no change) to check that the CI breaking is not due to some changes in another PR. Would some of you have an idea of what's causing the error? @SamuelGabriel @davidotte maybe? Otherwise I'll investigate. Maybe something changed in the CI or something? I get no error when running the tests locally. |
No idea, I would also need to investigate 🤔 |
Could it be the mock server tabpfn-server-wjedmz7r5a-ez.a.run.app has changed? The restx library seems to be stable for a while |
Locally things are running, but not on the server |
Okay it seems to be due to version differences in the |
#51 should fix this |
Change Description
Try to be precise. You can additionally add comments to your PR, this might help the reviewer a lot.
If you used new dependencies: Did you add them to
requirements.txt
?Who did you ping on Mattermost to review your PR? Please ping that person again whenever you are ready for another review.
Breaking changes
If you made any breaking changes, please update the version number.
Breaking changes are totally fine, we just need to make sure to keep the users informed and the server in sync.
Does this PR break the API? If so, what is the corresponding server commit?
Does this PR break the user interface? If so, why?
Please do not mark comments/conversations as resolved unless you are the assigned reviewer. This helps maintain clarity during the review process.