Skip to content

another try on claude ci#109

Merged
allisoneer merged 1 commit intohumanlayer:mainfrom
allisoneer:another_code_ci_hotfix
May 20, 2025
Merged

another try on claude ci#109
allisoneer merged 1 commit intohumanlayer:mainfrom
allisoneer:another_code_ci_hotfix

Conversation

@allisoneer
Copy link
Copy Markdown
Contributor

@allisoneer allisoneer commented May 20, 2025

CI

Important

Remove max_turns configuration from Claude GitHub workflows to simplify setup.

  • Workflows:
    • Remove max_turns configuration from claude-comment-response.yml and claude-pr-creation.yml, simplifying the setup by eliminating the conversation turn limit.

This description was created by Ellipsis for eda6ef8. You can customize this summary. It will automatically update as commits are pushed.

@allisoneer allisoneer changed the title another try another try on claude ci May 20, 2025
Copy link
Copy Markdown
Contributor

@ellipsis-dev ellipsis-dev bot left a comment

Choose a reason for hiding this comment

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

Important

Looks good to me! 👍

Reviewed everything up to eda6ef8 in 29 seconds. Click for details.
  • Reviewed 26 lines of code in 2 files
  • Skipped 0 files when reviewing.
  • Skipped posting 2 draft comments. View those below.
  • Modify your settings and rules to customize what types of comments Ellipsis leaves. And don't forget to react with 👍 or 👎 to teach Ellipsis.
1. .github/workflows/claude-comment-response.yml:50
  • Draft comment:
    Removal of max_turns may lead to unbounded conversation loops. Confirm if default limits are safe.
  • Reason this comment was not posted:
    Confidence changes required: 50% <= threshold 50% None
2. .github/workflows/claude-pr-creation.yml:62
  • Draft comment:
    Ensure that removal of max_turns (was 20) doesn't allow excessive interactions; verify intended default behavior.
  • Reason this comment was not posted:
    Confidence changes required: 50% <= threshold 50% None

Workflow ID: wflow_ZpOw4h6cnAEgEzcT

You can customize Ellipsis by changing your verbosity settings, reacting with 👍 or 👎, replying to comments, or adding code review rules.

@allisoneer allisoneer merged commit 49255d6 into humanlayer:main May 20, 2025
1 of 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.

2 participants