Skip to content

remove workflow (not real)#112

Merged
allisoneer merged 2 commits intohumanlayer:mainfrom
allisoneer:ci_claude2
May 20, 2025
Merged

remove workflow (not real)#112
allisoneer merged 2 commits intohumanlayer:mainfrom
allisoneer:ci_claude2

Conversation

@allisoneer
Copy link
Contributor

@allisoneer allisoneer commented May 20, 2025

Important

Remove workflow: write permission from three GitHub Actions workflows to enhance security.

  • Permissions:
    • Remove workflow: write permission from .github/workflows/claude-code-review.yml, .github/workflows/claude-comment-response.yml, and .github/workflows/claude-pr-creation.yml.

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

Copy link
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 fe781d3 in 41 seconds. Click for details.
  • Reviewed 36 lines of code in 3 files
  • Skipped 0 files when reviewing.
  • Skipped posting 3 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-code-review.yml:16
  • Draft comment:
    Removed the 'workflow: write' permission. Ensure that the CLAUDE action doesn't need this for any workflow operations.
  • Reason this comment was not posted:
    Confidence changes required: 33% <= threshold 50% None
2. .github/workflows/claude-comment-response.yml:16
  • Draft comment:
    Removal of 'workflow: write' permission seems intended. Confirm it doesn't impact any actions in this workflow.
  • Reason this comment was not posted:
    Confidence changes required: 33% <= threshold 50% None
3. .github/workflows/claude-pr-creation.yml:22
  • Draft comment:
    Removed 'workflow: write' permission in the PR creation workflow. Verify that no functionality relies on workflow modifications.
  • Reason this comment was not posted:
    Confidence changes required: 33% <= threshold 50% None

Workflow ID: wflow_PDTFgECo4RQUar1c

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

@allisoneer
Copy link
Contributor Author

merging due to not impacting codebase and ci iteration

@allisoneer allisoneer merged commit bdd8d9b into humanlayer:main May 20, 2025
1 check 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

Comments