Skip to content

change api key name #76

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
hamishfagg merged 1 commit into from
May 19, 2025
Merged

change api key name #76

hamishfagg merged 1 commit into from
May 19, 2025

Conversation

@hamishfagg
Copy link
Contributor

@hamishfagg hamishfagg commented May 13, 2025 •
edited
Loading

Makes the secret name more descriptive, and updates some URLs

@trag-bot
Copy link

trag-bot bot commented May 13, 2025

@trag-bot didn't find any issues in the code! ✅✨

@trag-bot
Copy link

trag-bot bot commented May 13, 2025

Pull request summary

  • Updated the GitHub Actions workflow to use MINDS_API_KEY instead of API_KEY for environment variable configuration.
  • Changed the default base URL in the get_client function from https://dev.mindsdb.com to https://dev.mdb.ai.
  • Ensured consistency in the naming of the API key across the workflow and the integration test file.

@hamishfagg hamishfagg merged commit daf47a4 into main May 19, 2025
6 checks passed
@hamishfagg hamishfagg deleted the fix/api_key_name branch May 19, 2025 21:37
@github-actions github-actions bot locked and limited conversation to collaborators May 19, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

@lucas-koontz lucas-koontz lucas-koontz approved these changes

@ZoranPandovski ZoranPandovski Awaiting requested review from ZoranPandovski

@tmichaeldb tmichaeldb Awaiting requested review from tmichaeldb

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@hamishfagg @lucas-koontz