Skip to content

Conversation

@blahblahasdf
Copy link
Collaborator

@blahblahasdf blahblahasdf commented Jan 30, 2026

I mistakenly used the Template format in the user prompt and we should be using the str.format directly.

Summary by CodeRabbit

  • Chores
    • Updated template placeholder syntax in configuration to use modern interpolation format.

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

Copy link
Contributor

@greptile-apps greptile-apps bot left a comment

Choose a reason for hiding this comment

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

No files reviewed, no comments

Edit Code Review Agent Settings | Greptile

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Jan 30, 2026

📝 Walkthrough

Walkthrough

Template placeholder syntax updated in a baseline configuration file from dollar-sign notation ($variable) to brace notation ({variable}), affecting three placeholder references in the Problem Description, Build command, and Context files blocks.

Changes

Cohort / File(s) Summary
Configuration Template Update
nemo_skills/prompt/config/compute-eval/baseline.yaml
Updated template placeholder syntax from $problem_prompt, $build_command, and $context_files_block to {problem_prompt}, {build_command}, and {context_files_block}.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

🚥 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 'Fix incorrect string format' accurately summarizes the main change: updating template placeholders from Template format ($variable) to Python str.format style ({variable}).
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.

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch compute-eval-prompt

Tip

🧪 Unit Test Generation v2 is now available!

We have significantly improved our unit test generation capabilities.

To enable: Add this to your .coderabbit.yaml configuration:

reviews:
  finishing_touches:
    unit_tests:
      enabled: true

Try it out by using the @coderabbitai generate unit tests command on your code files or under ✨ Finishing Touches on the walkthrough!

Have feedback? Share your thoughts on our Discord thread!


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

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

@gwarmstrong gwarmstrong enabled auto-merge (squash) January 30, 2026 00:34
@gwarmstrong gwarmstrong merged commit 3b44f02 into main Jan 30, 2026
5 checks passed
@gwarmstrong gwarmstrong deleted the compute-eval-prompt branch January 30, 2026 00:40
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.

3 participants