Skip to content

Conversation

raj-mistry-01
Copy link
Contributor

@king04aman

I have added that Voice assistant under Simple Assistant folder within my username as you have said.

I hope you that will help

I have briefly mentioned and made a readme how to set , what are features and all ..

Summary

Basic Python Voice Assistant.

I hope you will accept this PR under hactoberfest tag.

Thank you , Looking for the further contribution to the repo..

Copy link
Contributor

👋 @raj-mistry-01 👋

We're delighted to have your pull request! Please take a moment to check our contributing guidelines and ensure you've filled out the PR template for a smooth process. We will review it soon.

Copy link
Owner

@king04aman king04aman left a comment

Choose a reason for hiding this comment

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

Thanks for your contribution, Here are few changes, please consider changing them after that we are good to go 🥳

  • consider adding version in requirements.txt
  • File structure should be Simple Assistant/raj_mistry01/Your-Project-Files like:
    • Simple Assistant/raj_mistry01/README.md
    • Simple Assistant/raj_mistry01/main.py
    • Simple Assistant/raj_mistry01/requirements.txt

@raj-mistry-01
Copy link
Contributor Author

@king04aman

Now I have done that folder structure part , Sorry I thought under a other folder within username folder.

Thank you , Looking for the further contribution to repo..

Copy link
Owner

@king04aman king04aman left a comment

Choose a reason for hiding this comment

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

Thank you for your contributions towards this repo ❤️ . Happy Coding 🥳

@king04aman king04aman merged commit 8c6ae32 into king04aman:main Oct 23, 2024
Copy link
Contributor

👋 @king04aman 👋

🎉 You've just merged your pull request! We're excited to have you in our community. Keep up the fantastic contributions to the project!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants