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

implements #13 #36

Closed
wants to merge 0 commits into from
Closed

Conversation

ResourceHog
Copy link

No description provided.

@ResourceHog ResourceHog mentioned this pull request Apr 2, 2023
@alreadydone
Copy link

alreadydone commented Apr 2, 2023

The title could just be "implements #13" for automated linkification :)

@ResourceHog ResourceHog changed the title Implements https://github.com/Torantulino/Auto-GPT/issues/13 implements #13 Apr 2, 2023
@slavakurilyak
Copy link
Contributor

This pull request (#36) by @ResourceHog proposes to merge 5 commits into the master branch. The changes affect 4 files and result in a net addition of 84 lines and a deletion of 31 lines. The main changes include:

  1. scripts/commands.py: Minor import changes, with the addition of the print_to_console import from util.
  2. scripts/config.py: Added a new active_mode property and the corresponding setter method set_active_mode.
  3. scripts/main.py:
    • Removed the original print_to_console function and replaced it with the one imported from util.
    • Added a new ACTIVE_MODE argument to the Argument Enum.
    • Updated the parse_arguments function to handle the new active_mode configuration.
    • Added a new block of code to handle the active mode, which prompts the user to authorize or reject the command execution with a reason for rejection.
  4. scripts/util.py: This is a new file containing the print_to_console function, which has been moved from main.py.

It would be great to see an updated README.md to reflect "Active Mode".

@Torantulino
Copy link
Member

I want to implement this!
Mind bringing it up to date @ResourceHog?

@ResourceHog
Copy link
Author

Yeah, will get to it tonight. Basically just need to test it in the new docker environment before pushing it.

Provello
Provello previously approved these changes Apr 3, 2023
@ResourceHog
Copy link
Author

Power went out as I was working on this. Will have to wait till tomorrow.

waynehamadi
waynehamadi previously approved these changes Apr 4, 2023
@ResourceHog
Copy link
Author

ResourceHog commented Apr 4, 2023

It seems a bit redundant now that user input has been implemented in another pull request. Oh well, that happens sometimes!
#133

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.

None yet

6 participants