Skip to content
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.

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

Disable proxy for internal pull requests #4953

Merged
merged 1 commit into from Jul 12, 2023

Conversation

Pwuts
Copy link
Member

@Pwuts Pwuts commented Jul 12, 2023

Background

Changes

Pass OPENAI_API_TOKEN instead of using PROXY on internal PRs (so from a branch in the repo to another branch in the repo).

Documentation

x

Test Plan

CI

PR Quality Checklist

  • My pull request is atomic and focuses on a single change.
  • I have thoroughly tested my changes with multiple different prompts.
  • I have considered potential risks and mitigations for my changes.
  • I have documented my changes clearly and comprehensively.
  • I have not snuck in any "extra" small tweaks changes.

@Pwuts Pwuts requested a review from a team as a code owner July 12, 2023 15:25
@netlify
Copy link

netlify bot commented Jul 12, 2023

Deploy Preview for auto-gpt-docs canceled.

Name Link
🔨 Latest commit 7657edd
🔍 Latest deploy log https://app.netlify.com/sites/auto-gpt-docs/deploys/64aec5e6868230000871bfcf

@Pwuts Pwuts added ci testing API access Trouble with connecting to the API labels Jul 12, 2023
@Pwuts Pwuts merged commit 21c0cdc into master Jul 12, 2023
16 checks passed
@Pwuts Pwuts deleted the ci/disable-proxy-for-internal-pr branch July 12, 2023 15:50
@lc0rp lc0rp added this to the v0.4.5 Release milestone Jul 14, 2023
dayofthedave pushed a commit to dayofthedave/Auto-GPT that referenced this pull request Jul 17, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
API access Trouble with connecting to the API ci size/s testing
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

None yet

3 participants