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

Enabling/disabling none/all of plugins from .env file #4036

Merged
merged 5 commits into from May 19, 2023

Conversation

TrajanWJ
Copy link
Contributor

@TrajanWJ TrajanWJ commented May 8, 2023

Background

Changes

Small quality of life change to plugins.py that makes it so you can put "all" or "none" in the disabled or the enabled plugins and it will work.

Documentation

Small change.

Test Plan

PR Quality Checklist

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

Its litterally like half a line of code x2, just an ease of use thing I had in my local copy for a while.

@vercel
Copy link

vercel bot commented May 8, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

1 Ignored Deployment
Name Status Preview Comments Updated (UTC)
docs ⬜️ Ignored (Inspect) Visit Preview May 19, 2023 5:21pm

@github-actions github-actions bot added the size/s label May 8, 2023
@TrajanWJ
Copy link
Contributor Author

TrajanWJ commented May 8, 2023

Ive had this done with my copy for a while, just an ease of use thing for plugins. Not a big deal

@ntindle ntindle self-assigned this May 10, 2023
@codecov
Copy link

codecov bot commented May 10, 2023

Codecov Report

Patch coverage: 100.00% and no project coverage change.

Comparison is base (ee98641) 62.94% compared to head (2e01b33) 62.94%.

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #4036   +/-   ##
=======================================
  Coverage   62.94%   62.94%           
=======================================
  Files          74       74           
  Lines        3400     3400           
  Branches      495      495           
=======================================
  Hits         2140     2140           
  Misses       1111     1111           
  Partials      149      149           
Impacted Files Coverage Δ
autogpt/plugins.py 75.36% <100.00%> (ø)

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

Copy link
Member

@ntindle ntindle left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Add to docs/

@TrajanWJ
Copy link
Contributor Author

Add to docs

Will do ASAP, wanted to do some work on docs anyways. I think we should consider adding a "for devs" sections that explains the internals and architecture, or at least the principals that AUTOGPT uses to function.

ntindle
ntindle previously approved these changes May 18, 2023
@ntindle ntindle removed their assignment May 18, 2023
@ntindle ntindle added this to the v0.3.2 Release milestone May 18, 2023
@vercel vercel bot temporarily deployed to Preview May 18, 2023 06:40 Inactive
@k-boikov k-boikov changed the title Enabling/disabling none/all of plugins from .env file, small change to plugins.py Enabling/disabling none/all of plugins from .env file May 19, 2023
@k-boikov k-boikov merged commit ccc03c1 into Significant-Gravitas:master May 19, 2023
10 checks passed
ppetermann pushed a commit to ppetermann/Auto-GPT that referenced this pull request May 22, 2023
…avitas#4036)

Co-authored-by: Nicholas Tindle <nick@ntindle.com>
Co-authored-by: k-boikov <64261260+k-boikov@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

None yet

3 participants