Skip to content

registry: add moon (aqua:moonrepo/moon)#10140

Open
agdphd wants to merge 5 commits into
jdx:mainfrom
agdphd:patch-1
Open

registry: add moon (aqua:moonrepo/moon)#10140
agdphd wants to merge 5 commits into
jdx:mainfrom
agdphd:patch-1

Conversation

@agdphd
Copy link
Copy Markdown

@agdphd agdphd commented May 29, 2026

moon is a monorepo build system which my organization uses via Mise. Installing it via the Aqua backend works perfectly, so this file just aliases that for convenience.

moon is a monorepo build system which my organization uses via Mise. Installing it via the GitHub backend works perfectly, so this file just aliases that for convenience.
Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request adds a configuration file for moonrepo in registry/moon.toml. The feedback suggests improving the configuration by adding a test block and defining idiomatic_files to ensure proper validation in CI and auto-detection by mise.

Comment thread registry/moon.toml Outdated
@greptile-apps
Copy link
Copy Markdown
Contributor

greptile-apps Bot commented May 29, 2026

Greptile Summary

This PR adds moon (a monorepo build system from moonrepo) to the mise registry, pointing to the aqua:moonrepo/moon backend with an idiomatic_files entry for .moon/*.

  • The entry is functionally correct: backend, idiomatic files detection, and test command all look appropriate.
  • Two minor style inconsistencies exist vs the rest of the registry: the [test] block uses expanded TOML section syntax (the only entry to do so) and the expected value uses {{ version }} with spaces rather than the {{version}} form used in 535+ other entries.

Confidence Score: 5/5

A single new TOML registry entry with no logic or security implications; safe to merge.

The change is a straightforward registry addition. The backend, idiomatic-file detection, and test command are all correct. The only observations are minor style inconsistencies (template spacing and test block format) that do not affect runtime behaviour.

No files require special attention.

Important Files Changed

Filename Overview
registry/moon.toml Adds moon to the registry via aqua:moonrepo/moon; two style inconsistencies vs the rest of the registry: template variable uses {{ version }} (with spaces) instead of {{version}}, and the test block uses expanded [test] section syntax instead of the inline table used everywhere else.

Reviews (4): Last reviewed commit: "Switch to Aqua backend" | Re-trigger Greptile

Comment thread registry/moon.toml Outdated
Comment thread registry/moon.toml
@@ -0,0 +1,7 @@
description = "A build system and monorepo management tool for the web ecosystem, written in Rust"
backends = ["aqua:moonrepo/moon"]
idiomatic_files = [".moon/*"]
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

this doesn't make any sense

@agdphd agdphd changed the title registry: add moon (github:moonrepo/moon) registry: add moon (aqua:moonrepo/moon) May 29, 2026
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