Skip to content

Conversation

@kevinbackhouse
Copy link
Collaborator

I got AI_API_TOKEN and AI_API_ENDPOINT mixed up in the smoke test, which is why it's not working. I also made the same mistake in README.md.

Copilot AI review requested due to automatic review settings December 3, 2025 11:36
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 fixes a documentation error where the environment variable names AI_API_TOKEN and AI_API_ENDPOINT were incorrectly mixed up. The changes correct the documentation and workflow configuration to properly reference AI_API_TOKEN for authentication tokens, while AI_API_ENDPOINT remains as a separate variable for endpoint configuration.

Key Changes:

  • Corrected environment variable name in documentation from AI_API_ENDPOINT to AI_API_TOKEN
  • Fixed the smoketest workflow to use the correct AI_API_TOKEN secret

Reviewed changes

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

File Description
README.md Updated documentation and example configuration to correctly reference AI_API_TOKEN for the GitHub token
.github/workflows/smoketest.yaml Corrected environment variable in the smoke test workflow to use AI_API_TOKEN

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

@kevinbackhouse kevinbackhouse marked this pull request as ready for review December 3, 2025 11:48
@kevinbackhouse kevinbackhouse merged commit fbf550d into GitHubSecurityLab:main Dec 3, 2025
9 checks passed
@kevinbackhouse kevinbackhouse deleted the fix-smoketest branch December 3, 2025 16:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

2 participants