Skip to content

Conversation

@asimihsan
Copy link
Owner

No description provided.

@asimihsan asimihsan requested a review from Copilot June 12, 2025 10:54
@asimihsan asimihsan self-assigned this Jun 12, 2025
Copy link

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

This PR shifts the toolchain focus from Devbox to Mise by updating configuration files, documentation, and setup scripts. Key changes include:

  • Replacing simple defer calls with cleanup functions that log errors in WASM module bindings.
  • Introducing and updating various scripts and configuration files to utilize Mise for managing dependencies.
  • Removing outdated Devbox configuration and related files.

Reviewed Changes

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

Show a summary per file
File Description
template/{{"go-wasm" if include_go else "skip_go_wasm"}}/go/binding.go.jinja Updated WASM module cleanup to log errors instead of silently closing.
template/scripts/dev-setup.sh Added a setup script that installs Mise if missing.
template/mise.toml.jinja Removed in favor of the updated configuration in mise.toml.
template/mise.toml New configuration file specifying toolchain versions and tools.
template/devbox.json Removed outdated Devbox configuration.
template/Justfile.jinja Removed redundant devbox install command.
template/.envrc Removed outdated environment activation configuration.
scripts/test.sh Added test script for project generation and setup verification.
scripts/dev-setup.sh Added a standalone setup script for development environment.
mise.toml Updated toolchain configuration with new copier tool support.
devbox.json Removed obsolete Devbox configuration.
README.md Updated instructions to reflect the new focus on Mise.
.clinerules Removed outdated project guidelines.

@asimihsan asimihsan merged commit 6b9e685 into master Jun 12, 2025
5 checks passed
@asimihsan asimihsan deleted the asimihsan/revamp branch June 15, 2025 12:17
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.

2 participants