Skip to content

Use pixi instead of conda #287

Merged
johnstairs merged 10 commits into
mainfrom
johnstairs/skip-matlab
Jun 2, 2026
Merged

Use pixi instead of conda #287
johnstairs merged 10 commits into
mainfrom
johnstairs/skip-matlab

Conversation

@johnstairs
Copy link
Copy Markdown
Member

Migrating environment management from conda to pixi. Also skipping MATLAB installation on ARM64.

@johnstairs johnstairs requested a review from Copilot June 2, 2026 14:28
@johnstairs johnstairs changed the title Use Pixi instead of conda Use pixi instead of conda Jun 2, 2026
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Migrates the repository’s environment management from conda (environment.yml) to pixi (pixi.toml/pixi.lock), updating CI and the devcontainer to provision dependencies via Pixi and adding an option to skip MATLAB installation on ARM64.

Changes:

  • Replace environment.yml with Pixi workspace config (pixi.toml) and lockfile (pixi.lock), update docs/README accordingly.
  • Update GitHub Actions to use a new configure-environment composite action that sets up Pixi and Go using tooling/go.mod.
  • Rework devcontainer provisioning to install Pixi in the image and create the Pixi env in a persisted .pixi volume; add conditional MATLAB install and new post-* hooks.

Reviewed changes

Copilot reviewed 16 out of 18 changed files in this pull request and generated 3 comments.

Show a summary per file
File Description
tooling/go.mod Updates Go version used by CI/devcontainer tooling installs.
README.md Updates developer setup docs to reference Pixi instead of Conda.
pyrightconfig.json Expands Pyright excludes (node_modules, caches, dotdirs).
pixi.toml Defines Pixi workspace deps and a dev feature for local tooling.
pixi.lock Locks Pixi/conda package resolution for reproducible installs.
justfile Adds build dir cleaning and build-type-aware CMake configure recipes.
environment.yml Removes old Conda environment definition.
docs/cpp/installation.md Adds Pixi-based dependency installation guidance; keeps Conda alternative.
cpp/CMakeLists.txt Adjusts debug flags (adds -U_FORTIFY_SOURCE).
.vscode/c_cpp_properties.json Points VS Code C++ compiler path at Pixi-provided toolchain.
.gitignore Ignores .pixi/ (Pixi env/cache directory).
.github/workflows/ci.yml Switches CI to the new Pixi-based environment configuration.
.github/actions/configure-environment/action.yml Implements Pixi+Go setup for CI jobs.
.devcontainer/Dockerfile Replaces Conda install with Pixi install; makes MATLAB install conditional and skips on ARM64.
.devcontainer/devcontainer.json Adds Pixi volume mount and post-create/start hooks; updates tool paths to Pixi env.
.devcontainer/devcontainer.bashrc Activates Pixi dev environment for interactive shells.
.devcontainer/devcontainer-post-start.sh Ensures .bashrc sources the devcontainer bashrc after dotfiles apply.
.devcontainer/devcontainer-post-create.sh Creates the Pixi dev env, stable tool symlinks, and CMake Tools kits file.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread .devcontainer/devcontainer.json Outdated
Comment thread docs/cpp/installation.md Outdated
Comment thread .github/actions/configure-environment/action.yml Outdated
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
@johnstairs johnstairs marked this pull request as ready for review June 2, 2026 15:05
@johnstairs johnstairs requested a review from naegelejd June 2, 2026 15:05
Copy link
Copy Markdown
Contributor

@naegelejd naegelejd left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hooray 💯 🥳

Comment thread .github/actions/configure-environment/action.yml Outdated
@johnstairs johnstairs merged commit 7dc3130 into main Jun 2, 2026
14 checks passed
@johnstairs johnstairs deleted the johnstairs/skip-matlab branch June 2, 2026 18:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants