Add missing API Keys#267
Conversation
There was a problem hiding this comment.
Pull Request Overview
This PR adds API key environment variables for Anthropic, OpenAI, and Open Router services to the documentation workflow. The changes also include a minor whitespace cleanup.
- Added three API key environment variables (ANTHROPIC_API_KEY, OPEN_AI_API_KEY, OPEN_ROUTER_API_KEY) to the test environment
- Removed trailing whitespace from the "Setup Ruby" step name
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| env: | ||
| RAILS_ENV: test | ||
| RAILS_MASTER_KEY: ${{ secrets.RAILS_MASTER_KEY }} | ||
| ANTHROPIC_API_KEY: ANTHROPIC_API_KEY |
There was a problem hiding this comment.
The API key is set to the literal string "ANTHROPIC_API_KEY" instead of referencing a GitHub secret. It should be ${{ secrets.ANTHROPIC_API_KEY }} to properly inject the secret value, similar to how RAILS_MASTER_KEY is configured on line 58.
| ANTHROPIC_API_KEY: ANTHROPIC_API_KEY | |
| ANTHROPIC_API_KEY: ${{ secrets.ANTHROPIC_API_KEY }} |
| RAILS_ENV: test | ||
| RAILS_MASTER_KEY: ${{ secrets.RAILS_MASTER_KEY }} | ||
| ANTHROPIC_API_KEY: ANTHROPIC_API_KEY | ||
| OPEN_AI_API_KEY: OPEN_AI_API_KEY |
There was a problem hiding this comment.
The API key is set to the literal string "OPEN_AI_API_KEY" instead of referencing a GitHub secret. It should be ${{ secrets.OPEN_AI_API_KEY }} to properly inject the secret value, similar to how RAILS_MASTER_KEY is configured on line 58.
| OPEN_AI_API_KEY: OPEN_AI_API_KEY | |
| OPEN_AI_API_KEY: ${{ secrets.OPEN_AI_API_KEY }} |
| RAILS_MASTER_KEY: ${{ secrets.RAILS_MASTER_KEY }} | ||
| ANTHROPIC_API_KEY: ANTHROPIC_API_KEY | ||
| OPEN_AI_API_KEY: OPEN_AI_API_KEY | ||
| OPEN_ROUTER_API_KEY: OPEN_ROUTER_API_KEY |
There was a problem hiding this comment.
The API key is set to the literal string "OPEN_ROUTER_API_KEY" instead of referencing a GitHub secret. It should be ${{ secrets.OPEN_ROUTER_API_KEY }} to properly inject the secret value, similar to how RAILS_MASTER_KEY is configured on line 58.
| OPEN_ROUTER_API_KEY: OPEN_ROUTER_API_KEY | |
| OPEN_ROUTER_API_KEY: ${{ secrets.OPEN_ROUTER_API_KEY }} |
No description provided.