Skip to content

Conversation

@thatstoasty
Copy link
Contributor

@thatstoasty thatstoasty commented Oct 29, 2025

Checklist

  • My recipe.yaml file specifies which version(s) of MAX is compatible with my project (see here for an example). If not, my package is compatible with both 24.5 and 24.6.
  • License file is packaged (see here for an example).
  • Set the build number to 0 (for new packages, or if the version changed).
  • Bumped the build number (if the version is unchanged).

Copilot AI review requested due to automatic review settings October 29, 2025 16:47
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 updates the mist package recipe to version 25.6.0 and migrates from the legacy max dependency system to the newer mojo-compiler dependency system.

  • Version bump from 25.5.0 to 25.6.0
  • Migration from max to mojo-compiler dependencies
  • Addition of mojo_version context variable for version consistency

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

requirements:
host:
- max =25.5
- mojo-compiler ${{ mojo_version }}
Copy link

Copilot AI Oct 29, 2025

Choose a reason for hiding this comment

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

[nitpick] The host requirements section uses the template variable ${{ mojo_version }} while other recipes in the codebase (e.g., emberjson/recipe.yaml) use the hardcoded version =0.25.6 in the host section and the template variable in the build section. For consistency, consider using - mojo-compiler =0.25.6 in the host section, matching the pattern used in emberjson.

Suggested change
- mojo-compiler ${{ mojo_version }}
- mojo-compiler =0.25.6

Copilot uses AI. Check for mistakes.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants