Skip to content

Fix release slash command to use NOTES env var#36

Merged
agarcher merged 1 commit intomainfrom
fix-release-command
Jan 18, 2026
Merged

Fix release slash command to use NOTES env var#36
agarcher merged 1 commit intomainfrom
fix-release-command

Conversation

@agarcher
Copy link
Owner

@agarcher agarcher commented Jan 18, 2026

Summary

  • Fix the /release slash command to use the correct invocation format for the release script
  • The script expects NOTES="..." make release <type>, not make release <type> "..."

Test plan

  • Run /release command and verify it executes successfully

🤖 Generated with Claude Code

Summary by CodeRabbit

  • Chores
    • Updated the release notes invocation method to use environment variable configuration instead of inline argument passing.

✏️ Tip: You can customize this high-level summary in your review settings.

The release script expects notes via NOTES environment variable,
not as a positional argument.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@coderabbitai
Copy link

coderabbitai bot commented Jan 18, 2026

📝 Walkthrough

Walkthrough

The release command documentation has been updated to change how release notes are supplied to the release process. Instead of passing notes as an inline quoted argument, they are now provided via an environment variable NOTES that precedes the make release command invocation.

Changes

Cohort / File(s) Summary
Release Command Documentation
.claude/commands/release.md
Modified release invocation to accept release notes through the NOTES environment variable instead of an inline quoted argument, altering parameter passing methodology

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

🐰 A whisker of change, so neat and so small,
NOTES in the wind now, flowing through all,
Environment speaks where arguments once stood,
The release hops lighter, as documentation should! 📋✨

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title directly and specifically describes the main change: fixing the release slash command to use the NOTES environment variable instead of inline arguments.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.


Comment @coderabbitai help to get the list of available commands and usage tips.

@agarcher agarcher merged commit 65d114b into main Jan 18, 2026
5 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