Skip to content

fix: pass model via claude_args instead of invalid model input#47

Merged
Dorac merged 1 commit into
masterfrom
feat/claude-model-sonnet
Jun 22, 2026
Merged

fix: pass model via claude_args instead of invalid model input#47
Dorac merged 1 commit into
masterfrom
feat/claude-model-sonnet

Conversation

@Dorac

@Dorac Dorac commented Jun 22, 2026

Copy link
Copy Markdown
Contributor

Problem

PR #46 added model: 'claude-sonnet-4-6' as a direct input to claude-code-action, but the pinned version (9dd8b95a) does not support a model input — causing a warning and the setting being ignored.

Fix

Move model spec into claude_args via --model claude-sonnet-4-6, which is the correct way to pass it in this version.

PR Summary by Typo

Overview

This PR fixes an issue where the Claude model was incorrectly specified as a direct input instead of being passed through claude_args in the Dependabot auto-merge workflow.

Key Changes

  • The model input for the Claude action in .github/workflows/dependabot_auto_merge.yml has been removed.
  • The Claude model (claude-sonnet-4-6) is now correctly passed as an argument within the claude_args input.

Work Breakdown

Category Lines Changed
Churn 1 (50.0%)
Rework 1 (50.0%)
Total Changes 2
To turn off PR summary, please visit Notification settings.

@typo-app

typo-app Bot commented Jun 22, 2026

Copy link
Copy Markdown

Static Code Review 📊

✅ All quality checks passed!

@typo-app typo-app Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

AI Code Review 🤖

Files Reviewed: 1
Comments Added: 0
Lines of Code Analyzed: 3
Critical Issues: 0

PR Health: Excellent 🔥

Give 👍 or 👎 on each review comment to help us improve.

@Dorac Dorac merged commit 046c799 into master Jun 22, 2026
6 of 10 checks passed
@Dorac Dorac deleted the feat/claude-model-sonnet branch June 22, 2026 09:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants