Skip to content
This repository has been archived by the owner on Jun 9, 2024. It is now read-only.

Integrating riensen/Auto-GPT-Email-Plugin #24

Merged
merged 44 commits into from
Apr 22, 2023

Conversation

riensen
Copy link
Contributor

@riensen riensen commented Apr 22, 2023

This PR migrates my repo github.com/riensen/Auto-GPT-Email-Plugin

Torantulino and others added 30 commits April 11, 2023 12:48
…/makefile

Proposal: Consolidate Scripts and add Makefile

The Auto-GPT Email Plugin is an innovative and powerful plugin for the groundbreaking base software, Auto-GPT. Harnessing the capabilities of the latest Auto-GPT architecture, Auto-GPT aims to autonomously achieve any goal you set, pushing the boundaries of what is possible with artificial intelligence. This email plugin takes Auto-GPT to the next level by enabling it to send and read emails, opening up a world of exciting use cases.

[![GitHub Repo stars](https://img.shields.io/github/stars/riensen/Auto-GPT-Email-Plugin?style=social)](https://github.com/riensen/Auto-GPT-Email-Plugin/stargazers)
Copy link
Member

Choose a reason for hiding this comment

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

may want to repoint this to the main repo?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

done

This is the Auto-GPT email plugin.
"""

def __init__(self):
Copy link
Member

Choose a reason for hiding this comment

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

We should check if your required envs exist before we register

Copy link
Contributor Author

Choose a reason for hiding this comment

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

What are you trying to achieve? Do you want to throw an exception if the env vars do not exist?

Copy link
Member

Choose a reason for hiding this comment

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

I'd prefer it to not crash if they don't exist for all plugins

Copy link
Contributor Author

Choose a reason for hiding this comment

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

So in this case, it is working as intended right now. If they are not set, then it won't crash right now.
As opposed to the Twitter plugin. Here, if the env vars are not set, then it crashes with an exception.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

What we could do is not register the commands if the minimum required env vars are not set.

@ntindle
Copy link
Member

ntindle commented Apr 22, 2023

Fix the readme and we should be GTG here

@ntindle ntindle merged commit 9c63170 into Significant-Gravitas:master Apr 22, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

7 participants