Skip to content

Revert: Annuler les changements workflow CodeRabbit#25

Merged
EthanThePhoenix38 merged 1 commit intomainfrom
claude/configure-workflow-coderabbit-Gd8HS
Jan 15, 2026
Merged

Revert: Annuler les changements workflow CodeRabbit#25
EthanThePhoenix38 merged 1 commit intomainfrom
claude/configure-workflow-coderabbit-Gd8HS

Conversation

@EthanThePhoenix38
Copy link
Member

Retour à la configuration workflow originale.
Suppression du fichier coderabbit.yaml.

This reverts commit 59d6032.

Retour à la configuration workflow originale.
Suppression du fichier coderabbit.yaml.

This reverts commit 59d6032.
Copy link

@chatgpt-codex-connector chatgpt-codex-connector bot left a comment

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 58c78c1839

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment on lines 51 to +54
if ! git diff --cached --exit-code; then
UTC_DATE=$(date -u +'%a %b %d %H:%M:%S UTC %Y')
git commit -m "Updated AI-Pulse: $UTC_DATE [skip ci]"
git push origin main
git commit -m "Updated AI-Pulse: $UTC_DATE"
git push

Choose a reason for hiding this comment

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

P1 Badge Restore CI skip to prevent recursive workflow runs

Because this workflow triggers on push to main, removing the [skip ci] marker from the commit message means the job’s own git push will immediately retrigger itself. src/aggregator.js always rewrites the README with a fresh Last Update timestamp, so every run creates a new commit and the cycle repeats, potentially creating an endless cascade of runs and commits. This only happens when the workflow is triggered by a push (including the push it makes itself), but that is the default here.

Useful? React with 👍 / 👎.

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR reverts commit 59d6032, which had integrated CodeRabbit workflow automation. The revert removes the CodeRabbit configuration file and restores the original workflow configuration for the AI-Pulse auto-aggregator, including changes to git user configuration, commit messages, and authentication handling.

Changes:

  • Removed CodeRabbit configuration file (.github/coderabbit.yaml)
  • Reverted workflow permissions to only include 'contents: write'
  • Restored original git user configuration (ThePhoenixAgency instead of github-actions[bot])
  • Removed explicit token and credential persistence from checkout action
  • Simplified commit message and git push command

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 3 comments.

File Description
.github/workflows/update-ai-pulse.yml Reverted workflow configuration to remove CodeRabbit-specific changes including permissions, checkout parameters, git configuration, and commit message patterns
.github/coderabbit.yaml Deleted CodeRabbit configuration file that defined auto-review and auto-approve settings

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

git commit -m "Updated AI-Pulse: $UTC_DATE [skip ci]"
git push origin main
git commit -m "Updated AI-Pulse: $UTC_DATE"
git push
Copy link

Copilot AI Jan 15, 2026

Choose a reason for hiding this comment

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

The git push command without explicit branch specification may fail or push to an unexpected branch. When the workflow runs on a schedule or via workflow_dispatch, it may be in a detached HEAD state. Consider explicitly specifying the branch and remote as 'git push origin main' to ensure the changes are pushed to the correct branch consistently.

Suggested change
git push
git push origin main

Copilot uses AI. Check for mistakes.
UTC_DATE=$(date -u +'%a %b %d %H:%M:%S UTC %Y')
git commit -m "Updated AI-Pulse: $UTC_DATE [skip ci]"
git push origin main
git commit -m "Updated AI-Pulse: $UTC_DATE"
Copy link

Copilot AI Jan 15, 2026

Choose a reason for hiding this comment

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

The commit message no longer includes '[skip ci]' which was present in the reverted version. This means every automated commit will trigger the workflow again due to the 'on: push: branches: - main' trigger, potentially creating an infinite loop. Consider adding '[skip ci]' back to the commit message to prevent the workflow from triggering itself.

Suggested change
git commit -m "Updated AI-Pulse: $UTC_DATE"
git commit -m "Updated AI-Pulse: $UTC_DATE [skip ci]"

Copilot uses AI. Check for mistakes.
@@ -20,9 +18,6 @@ jobs:
steps:
- name: Check out repository
uses: actions/checkout@v3
Copy link

Copilot AI Jan 15, 2026

Choose a reason for hiding this comment

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

Without explicitly providing a token to the checkout action or persisting credentials, the subsequent git push may fail with authentication errors. The default GITHUB_TOKEN provided to workflows may not be automatically available for git operations. Consider either adding 'token: ${{ secrets.GITHUB_TOKEN }}' to the checkout action or ensuring that authentication is properly configured for the git push operation.

Suggested change
uses: actions/checkout@v3
uses: actions/checkout@v3
with:
token: ${{ secrets.GITHUB_TOKEN }}

Copilot uses AI. Check for mistakes.
@EthanThePhoenix38 EthanThePhoenix38 merged commit c7e014b into main Jan 15, 2026
10 checks passed
@EthanThePhoenix38 EthanThePhoenix38 deleted the claude/configure-workflow-coderabbit-Gd8HS branch January 15, 2026 08:33
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.

3 participants