Skip to content

update: update readme#1714

Merged
cubxxw merged 4 commits intoopenimsdk:mainfrom
skiffer-git:update-readme-1713
Jan 8, 2024
Merged

update: update readme#1714
cubxxw merged 4 commits intoopenimsdk:mainfrom
skiffer-git:update-readme-1713

Conversation

@skiffer-git
Copy link
Copy Markdown
Contributor


🔍 What type of PR is this?

👀 What this PR does / why we need it:

  • My pull request adheres to the code style of this project
  • My code requires changes to the documentation
  • I have updated the documentation as required
  • All the tests have passed

🅰 Which issue(s) this PR fixes:

Fixes #1713

📝 Special notes for your reviewer:

🎯 Describe how to verify it

📑 Additional documentation e.g., RFC, notion, Google docs, usage docs, etc.:

@pull-request-size pull-request-size bot added the size/L Denotes a PR that changes 100-499 lines, ignoring generated files. label Jan 8, 2024
@sweep-ai-deprecated
Copy link
Copy Markdown
Contributor

Apply Sweep Rules to your PR?

  • Apply: All new business logic should have corresponding unit tests.
  • Apply: Refactor large functions to be more modular.
  • Apply: Add docstrings to all functions and file headers.

FGadvancer
FGadvancer previously approved these changes Jan 8, 2024
@cubxxw cubxxw enabled auto-merge January 8, 2024 13:39
@cubxxw cubxxw added this pull request to the merge queue Jan 8, 2024
Merged via the queue into openimsdk:main with commit 3ff5888 Jan 8, 2024
@github-actions github-actions bot added this to the v3.1 milestone Jan 8, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

size/L Denotes a PR that changes 100-499 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Bug: "Update the document, regarding the deployment section

3 participants