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

Make ci happy #91

Merged
merged 4 commits into from
May 24, 2023
Merged

Make ci happy #91

merged 4 commits into from
May 24, 2023

Conversation

yihong0618
Copy link
Contributor

This PR do these things

  • lint all code use black and isort
  • delete all unused imports to make code clean
  • use black in GItHub Actions
  • add doc to these when contribute

@csunny

@@ -67,7 +70,7 @@ def append_message(self, role, message):

def to_gradio_chatbot(self):
ret = []
for i, (role, msg) in enumerate(self.messages[self.offset:]):
for i, (role, msg) in enumerate(self.messages[self.offset :]):
Copy link
Collaborator

Choose a reason for hiding this comment

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

This space for why?

Copy link
Collaborator

@csunny csunny left a comment

Choose a reason for hiding this comment

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

approve

@csunny csunny merged commit aed3e8f into eosphoros-ai:main May 24, 2023
yhjun1026 pushed a commit that referenced this pull request Sep 14, 2023
This PR do these things

- lint all code use `black` and `isort`
- delete all unused imports to make code clean
- use black in GItHub Actions
- add doc to these when contribute

@csunny
yhjun1026 pushed a commit that referenced this pull request Sep 14, 2023
This PR do these things

- lint all code use `black` and `isort`
- delete all unused imports to make code clean
- use black in GItHub Actions
- add doc to these when contribute

@csunny
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