Skip to content
This repository was archived by the owner on Jun 17, 2026. It is now read-only.

ci: Add mooncakes publish workflow (v0.1.2), resolve MoonBit fmt drift, and enforce pre-commit formatting - #5

Merged
yufeiminds merged 4 commits into
mainfrom
copilot/fix-publish-issues-mooncakes
Apr 16, 2026
Merged

yufeiminds merged 4 commits into
mainfrom
copilot/fix-publish-issues-mooncakes

Conversation

Copilot AI commented Apr 16, 2026 •

Copy link
Copy Markdown
  • Review new PR comment and identify referenced failing CI job
  • Inspect workflow run/job logs for the reported moon fmt --check failure
  • Apply minimal formatting fix based on CI diff output
  • Run targeted formatting validation locally (blocked: moon binary not available in this environment)
  • Run final parallel validation (code review + CodeQL)
  • Reply to PR comment with fixing commit hash

Copilot AI linked an issue Apr 16, 2026 that may be closed by this pull request
Copilot AI changed the title [WIP] Fix publish issues for mooncakes Add mooncakes publish workflow (v0.1.2) and resolve MoonBit fmt drift in cargo tests Apr 16, 2026
Copilot AI requested a review from yufeiminds April 16, 2026 09:52
@yufeiminds
yufeiminds marked this pull request as ready for review April 16, 2026 09:55
@yufeiminds

Copy link
Copy Markdown
Contributor

@copilot Please fix the format error in CI and add the format to the pre-commit.

Agent-Logs-Url: https://github.com/cogna-dev/sbom/sessions/6b455b8c-6bc9-40ca-b642-8ca42e378f63

Co-authored-by: yufeiminds <9150374+yufeiminds@users.noreply.github.com>

Copilot AI commented Apr 16, 2026

Copy link
Copy Markdown
Author

@copilot Please fix the format error in CI and add the format to the pre-commit.

Implemented in 3f9e0d0.

  • Fixed remaining CI format diffs in src/cmd/moon.pkg, src/packages/go/go.mbt, and tests/cargo/cargo_test.mbt to match moon fmt --check output.
  • Added pre-commit formatting enforcement via .pre-commit-config.yaml with a local moon fmt --check hook.

Copilot AI changed the title Add mooncakes publish workflow (v0.1.2) and resolve MoonBit fmt drift in cargo tests Add mooncakes publish workflow (v0.1.2), resolve MoonBit fmt drift, and enforce pre-commit formatting Apr 16, 2026
@yufeiminds

Copy link
Copy Markdown
Contributor

@copilot moon fmt --check still error in https://github.com/cogna-dev/sbom/actions/runs/24504320945/job/71619038867?pr=5

Agent-Logs-Url: https://github.com/cogna-dev/sbom/sessions/67a0d1a2-ae4a-4c6f-b716-42fe411de67c

Co-authored-by: yufeiminds <9150374+yufeiminds@users.noreply.github.com>

Copilot AI commented Apr 16, 2026

Copy link
Copy Markdown
Author

@copilot moon fmt --check still error in https://github.com/cogna-dev/sbom/actions/runs/24504320945/job/71619038867?pr=5

Fixed in 07ccc70.

I checked the referenced job log and applied the exact remaining fmt change in src/cmd/moon.pkg (added the blank line required by moon fmt --check between the import block and options(...)).

Screenshot: N/A (no UI changes in this PR).

@yufeiminds yufeiminds changed the title Add mooncakes publish workflow (v0.1.2), resolve MoonBit fmt drift, and enforce pre-commit formatting ci: Add mooncakes publish workflow (v0.1.2), resolve MoonBit fmt drift, and enforce pre-commit formatting Apr 16, 2026
@yufeiminds
yufeiminds merged commit a15c238 into main Apr 16, 2026
2 checks passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

fix: Publish issues for mooncakes

2 participants