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

Windows Compatibility and Dependency Updates: Bash/Makefile Replacement, Error Fixes, and Requirements Update #103

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

joaocarlosleme
Copy link

@joaocarlosleme joaocarlosleme commented Jul 22, 2023

This pull request enhances cross-platform compatibility, primarily targeting Windows users, resolves some errors, and updates some dependencies:

  1. Bash to Batch Conversion: Replaced the ingest.sh bash script with a Windows-compatible ingest.bat batch file, facilitating execution of the script for Windows users.

  2. Makefile to Python Conversion: Replaced the Makefile with a Python script to start the server and format the code. This change allows all users, regardless of their platform, to start the server and format code without the need for the make command.

  3. Added environment variables: Added .env file to store OpenAI API Key and updated code to use it.

  4. Error Fixes: Addressed and fixed errors in the codebase to enable it to run, including updating requirements.txt.

  5. Updated README: To reflect changes and added directions to windows users.

These are the changes required to get it running on Windows and enhance the productivity of developers across all platforms. Please review the changes and provide feedback.

Changes required to run properly, fixing errors and making it windows compatible.
Update README with instructions about setting up ENV file
Adjusted URL since the previous one was being reported as not working on MAC/Linux as well.
Adjusted URL since the previous one was being reported as not working on MAC/Linux as well.
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.

1 participant