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

Cleanup azure parameters in env.template and remove unused env in config.py #1269

Merged

Conversation

suensummit
Copy link
Contributor

Background

There are some unused azure environment variables left in the env.template as they are already moved to azure.yaml.template.

Changes

  • Cleanup Azure related env from env.template.
  • Remove unused azure parameters from config.py, also fix the yaml loader syntax with some preset values.

Documentation

There are notes in env.template and the codes in config.py is clear enough.

Test Plan

Tested with azure configurations.

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

@suensummit
Copy link
Contributor Author

suensummit commented Apr 14, 2023

@richbeales @sweetlilmre please take a look thanks.

@suensummit suensummit force-pushed the cleanup-unused-azure-env branch from e043c85 to 0ff1f1c Compare April 15, 2023 03:29
@suensummit
Copy link
Contributor Author

conflicts resolved, please review while available, thanks @nponeccop @sweetlilmre @richbeales @Torantulino

@nponeccop
Copy link
Contributor

@suensummit There are conflicts again

@suensummit suensummit force-pushed the cleanup-unused-azure-env branch from 0ff1f1c to a0de386 Compare April 15, 2023 14:47
@suensummit
Copy link
Contributor Author

@suensummit There are conflicts again

Resolved

@richbeales richbeales merged commit 8c4b985 into Significant-Gravitas:master Apr 15, 2023
@nponeccop nponeccop mentioned this pull request Apr 15, 2023
1 task
sindlinger pushed a commit to Orgsindlinger/Auto-GPT-WebUI that referenced this pull request Sep 25, 2024
…unused-azure-env

Cleanup azure parameters in env.template and remove unused env in config.py
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants