Skip to content

Make ci happy#91

Merged
csunny merged 4 commits into
eosphoros-ai:mainfrom
yihong0618:make_ci_happy
May 24, 2023
Merged

Make ci happy#91
csunny merged 4 commits into
eosphoros-ai:mainfrom
yihong0618:make_ci_happy

Conversation

@yihong0618
Copy link
Copy Markdown
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

Comment thread pilot/conversation.py
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
Copy Markdown
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
Copy Markdown
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