chore: use org-level TG_TOKEN secret#245
Conversation
STRATEGY_PLUGIN_ALERT_TELEGRAM_BOT_TOKEN secret → org-level TG_TOKEN Co-Authored-By: Claude <noreply@anthropic.com>
|
Bugbot is not enabled for your account, so this pull request was not reviewed. Enable Bugbot in the Cursor dashboard to get automatic reviews on future PRs. |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 029a477f83
ℹ️ 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".
| STRATEGY_PLUGIN_ALERT_PUSH_APP_TOKEN: ${{ secrets.STRATEGY_PLUGIN_ALERT_PUSH_APP_TOKEN }} | ||
| STRATEGY_PLUGIN_ALERT_PUSH_ACCESS_TOKEN: ${{ secrets.STRATEGY_PLUGIN_ALERT_PUSH_ACCESS_TOKEN }} | ||
| STRATEGY_PLUGIN_ALERT_TELEGRAM_BOT_TOKEN: ${{ secrets.STRATEGY_PLUGIN_ALERT_TELEGRAM_BOT_TOKEN }} | ||
| STRATEGY_PLUGIN_ALERT_TELEGRAM_BOT_TOKEN: ${{ secrets.TG_TOKEN }} |
There was a problem hiding this comment.
Update the workflow smoke test for TG_TOKEN
When tests/test_sync_cloud_run_env_workflow.sh is run, its line 97 still greps for STRATEGY_PLUGIN_ALERT_TELEGRAM_BOT_TOKEN: ${{ secrets.STRATEGY_PLUGIN_ALERT_TELEGRAM_BOT_TOKEN }}, so this changed workflow line makes the smoke test exit with status 1 even though the workflow now intentionally uses secrets.TG_TOKEN. Please update the assertion to the new secret name so the workflow test remains usable.
Useful? React with 👍 / 👎.
1 行:secrets.STRATEGY_PLUGIN_ALERT_TELEGRAM_BOT_TOKEN → secrets.TG_TOKEN。环境变量名不变。