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

Refine English README and Change Dependency Installation Method #130

Merged
merged 1 commit into from
Nov 12, 2023

Conversation

qidanrui
Copy link
Collaborator

@qidanrui qidanrui commented Nov 12, 2023

This PR makes the following changes:

  • Refine the English readme

  • Change the installation method of DB-GPT-Hub from requirement.txt to poetry install, the advantages of Python Poetry includes:

    1. Prepare for building an automated workflow, which includes type checking, building, and testing that make it convenient to review and merge contributed PRs after thorough evaluation.
Screen Shot 2023-11-12 at 10 21 31 AM
  1. Convenient for adding and removing dependencies. For example, if you want to add a special version of black package, just run poetry add black==0.1.0. If you want to remove it, just run poetry remove black

  2. Prepare for the automated releasing/publishing workflow to the PyPI (Python Package Index) and Package Repository of Anaconda

Screen Shot 2023-11-12 at 10 26 43 AM
  1. Enable automatic testing of our DB-GPT-Hub project on different Operating Systems and Python Environments.
Screen Shot 2023-11-12 at 10 38 35 AM
  • Refine code style with black
    • Just run poetry run black dbgpt_hub
  • Ignore .vscode folder in .gitignore

…fine code style with black 4. ignore vscode file
Copy link
Member

@wangzaistone wangzaistone left a comment

Choose a reason for hiding this comment

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

Great work !

@wangzaistone wangzaistone merged commit 8f2836e into eosphoros-ai:main Nov 12, 2023
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