Skip to content

Recommend unnecessary annotation warning in MoonBit workflow - #43

Merged
Yoorkin merged 1 commit into
mainfrom
codex/moonbit-unnecessary-annotation-warning
Apr 28, 2026
Merged

Recommend unnecessary annotation warning in MoonBit workflow#43
Yoorkin merged 1 commit into
mainfrom
codex/moonbit-unnecessary-annotation-warning

Conversation

@Yoorkin

@Yoorkin Yoorkin commented Apr 27, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Recommend moon check --warn-list +unnecessary_annotation in the general MoonBit agent validation loop.
  • Note that +73 is the equivalent numeric warning form and that it catches redundant annotations and over-qualified constructors early.

Validation

  • git diff --check
  • moon test moonbit-agent-guide/SKILL.mbt.md

@Yoorkin
Yoorkin force-pushed the codex/moonbit-unnecessary-annotation-warning branch 2 times, most recently from 703ec6c to bf64617 Compare April 27, 2026 07:07
@Yoorkin
Yoorkin force-pushed the codex/moonbit-unnecessary-annotation-warning branch from bf64617 to 1ebc072 Compare April 27, 2026 07:09
@Yoorkin
Yoorkin requested a review from bobzhang April 27, 2026 07:09
@Yoorkin
Yoorkin marked this pull request as ready for review April 27, 2026 07:09

@devin-ai-integration devin-ai-integration Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no potential bugs to report.

View in Devin Review to see 1 additional finding.

Open in Devin Review

@Yoorkin
Yoorkin merged commit 723d45c into main Apr 28, 2026
2 checks passed
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.

1 participant