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

Base branch for CI/env fixes #296

Merged
merged 48 commits into from
Dec 31, 2022
Merged

Base branch for CI/env fixes #296

merged 48 commits into from
Dec 31, 2022

Conversation

dominiquesydow
Copy link
Collaborator

@dominiquesydow dominiquesydow commented Dec 27, 2022

Description

Current teachopencadd has a couple of dependency conflicts and deps-unrelated pytest fails.
This base branch shall be used to get a fully passing CI, while fixing all the different bits and pieces in smaller PRs.

Todos

Notes: After this PR's merge

...continue with TODOs in #312

Status

  • Ready to go

@dominiquesydow dominiquesydow self-assigned this Dec 27, 2022
@dominiquesydow dominiquesydow added bug Something isn't working dependencies Update/fix dependencies labels Dec 27, 2022
@dominiquesydow dominiquesydow linked an issue Dec 28, 2022 that may be closed by this pull request
T018: Remove nbval check for complex cell output
@review-notebook-app
Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

@dominiquesydow dominiquesydow linked an issue Dec 28, 2022 that may be closed by this pull request
@dominiquesydow dominiquesydow mentioned this pull request Dec 31, 2022
3 tasks
@dominiquesydow
Copy link
Collaborator Author

CI is passing for all notebooks except for T019, which we will tackle in a follow-up PR, see #308.

@dominiquesydow dominiquesydow merged commit 66a8283 into master Dec 31, 2022
@dominiquesydow dominiquesydow deleted the base-ci-env-fix branch December 31, 2022 02:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment