Skip to content

fix build#14

Merged
newbe36524 merged 1 commit intomainfrom
fixbuild
Feb 10, 2026
Merged

fix build#14
newbe36524 merged 1 commit intomainfrom
fixbuild

Conversation

@newbe36524
Copy link
Contributor

@newbe36524 newbe36524 commented Feb 10, 2026

Summary by CodeRabbit

  • Chores
    • Updated GitHub Actions workflow configuration for Azure Sync to require explicit Feishu webhook URL input
    • Changed secret handling in the Azure Sync Notification step to use explicit environment variable instead of inheriting all secrets

@coderabbitai
Copy link

coderabbitai bot commented Feb 10, 2026

Caution

Review failed

The pull request is closed.

📝 Walkthrough

Walkthrough

Modified GitHub Actions workflow configuration in sync-azure-storage.yml to add a required FEISHU_WEBHOOK_URL secret input to the workflow-call trigger and explicitly pass it as an environment variable to the notification job, replacing the previous secrets: inherit approach.

Changes

Cohort / File(s) Summary
GitHub Actions Workflow Configuration
.github/workflows/sync-azure-storage.yml
Added required FEISHU_WEBHOOK_URL secret input to on.workflow_call.secrets. Changed Azure Sync Notification job from using secrets: inherit to explicitly mapping the secret via FEISHU_WEBHOOK_URL environment variable.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related PRs

  • sync-azure if release success #7: Also modifies .github/workflows/sync-azure-storage.yml and updates how secrets and workflow inputs are handled in this workflow.

Poem

🐰 A secret made explicit, no more inherit's might,
The webhook URL now declared, shining clear and bright,
We hopped from implicit trust to transparency's way,
The workflow sings more sweetly now, in a cleaner display! 🌿✨

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

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

@newbe36524 newbe36524 merged commit 496ed2e into main Feb 10, 2026
2 of 3 checks passed
@newbe36524 newbe36524 deleted the fixbuild branch February 10, 2026 11:20
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