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

bot: implement basic features in bot #1

Merged
merged 1 commit into from
Jun 7, 2024

Conversation

ksauraj
Copy link
Member

@ksauraj ksauraj commented Jun 7, 2024

Description

This commit adds multiple features to bot

Screenshots

image
image
image

Checklist

  • My pull request has a descriptive title (not a vague title like Update index.md).
  • My pull request targets the default branch of the repository (main or master).
  • My commit messages follow [best practices][best_practices].
  • My code follows the established code style of the repository.
  • I added or updated tests for the changes I made (if applicable).
  • I added or updated documentation (if applicable).
  • I tried running the project locally and verified that there are no
    visible errors.

@Utkarsh4517 Utkarsh4517 requested a review from Mr-Sunglasses June 7, 2024 13:29
echo -e "\e[1;34m$*\e[0m"
}

TOKEN=""
Copy link
Member

Choose a reason for hiding this comment

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

is there any way we can use .env here and fetch all these from there.

Copy link
Member Author

Choose a reason for hiding this comment

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

Actually we can do this but we have something similar already implemented, like currently this init.sh script take all variable and store in a file called .token.sh and whenever bot in started with ./bot.sh it takes configs from this file, so in short we just need to rename the .token.sh to config.env

@Mr-Sunglasses Mr-Sunglasses merged commit 9ff6517 into FOSS-Community:main Jun 7, 2024
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.

2 participants