Chore/dev env fixes#1279
Conversation
github-actions
Bot
added
bug
| // Uncomment to connect as an existing user other than the container default. More info: https://aka.ms/dev-containers-non-root. | ||
| // "remoteUser": "devcontainer" | ||
| } | ||
| "image": "mcr.microsoft.com/devcontainers/typescript-node:0-18", |
There was a problem hiding this comment.
we just need pixi and docker now
There was a problem hiding this comment.
nodejs is in here for LLM tooling, we can remove if that's not needed
| #! /usr/bin/bash | ||
| # This script registers dot plugins so that we can use graphviz | ||
| # to write png images | ||
| dot -c No newline at end of file |
There was a problem hiding this comment.
the dot tool used by graphviz uses a post-installation step to register different file format backends. this script performs that step. without this, the graphviz-related tests will fail in a fresh environment.
| "setuptools", | ||
| ] | ||
| requires-python = ">=3.9,<4.0" | ||
| requires-python = ">=3.9,<3.14" |
There was a problem hiding this comment.
linux tests fail on python 3.14 due to attempts to pickle things that should not be pickled (locks). Fixing this architecturally requires quite a bit more work so we avoid python 3.14 for now.
There was a problem hiding this comment.
3.9 has reached its end of life.
There was a problem hiding this comment.
👍 then lets move to 3.11
| hooks: | ||
| - id: black | ||
| args: ["--check", "-v", "src", "tests", "--diff"] # --required-version is conflicting with pre-commit | ||
| args: ["-v", "src", "tests", "--diff"] # --required-version is conflicting with pre-commit |
There was a problem hiding this comment.
makes pre-commit much more useful by applying the formatting changes
2 participants
Applies more fixes to the developer environment. I will add explanatory comments to new / altered files.