Skip to content

Conversation

@simrat-code
Copy link

There is double 'can can' in the documentation.

Why are these changes needed?

typo-fix in the documentation.

Related issue number

Checks

There is double 'can can' in the documentation.
@ekzhu ekzhu added the 0.2 Issues which are related to the pre 0.4 codebase label Mar 4, 2025
@ekzhu
Copy link
Contributor

ekzhu commented Mar 4, 2025

Pls fix formatting

@rysweet
Copy link
Collaborator

rysweet commented Mar 4, 2025

Pls fix formatting

@simrat-code

@simrat-code
Copy link
Author

Hi @rysweet , which formatting you are referring to, can you direct/guide me? I only touch one line in llm_configuration.ipynb

@rysweet
Copy link
Collaborator

rysweet commented Mar 13, 2025

Have a look at the contrib guides - you need to run "poe format" I think

@ekzhu
Copy link
Contributor

ekzhu commented Mar 14, 2025

@simrat-code in this case this is 0.2 branch. Please use pre-commit --all-files to fix format

Copy link
Author

@simrat-code simrat-code left a comment

Choose a reason for hiding this comment

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

fix

  1. double 'can can' typo
  2. format issue, while execute pre-commit

@simrat-code
Copy link
Author

@simrat-code in this case this is 0.2 branch. Please use pre-commit --all-files to fix format

thanks :) I fixed the issue while executing 'pre-commit'

@ekzhu
Copy link
Contributor

ekzhu commented Mar 21, 2025

Merging the change. CI error is due to openai API change.

@ekzhu ekzhu merged commit 0806b5d into microsoft:0.2 Mar 21, 2025
12 of 18 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

0.2 Issues which are related to the pre 0.4 codebase

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants