Skip to content

feat(ux): better readme#20

Merged
shiqian-su merged 3 commits intomainfrom
dev-better-readme
Aug 25, 2025
Merged

feat(ux): better readme#20
shiqian-su merged 3 commits intomainfrom
dev-better-readme

Conversation

@ntudy
Copy link
Contributor

@ntudy ntudy commented Aug 24, 2025

Describe this PR

Checklist for PR

Must Do

  • Write a good PR title and description, i.e. feat(agent): add pdf tool via mcp, perf: make llm client async and fix(utils): load custom config via importlib etc. CI job check-pr-title enforces Angular commit message format to PR title.
  • Run make precommit locally. CI job lint enforce ruff default format/lint rules on all new codes.
  • Run make pytest. Check test summary (located at report.html) and coverage report (located at htmlcov/index.html) on new codes.

Nice To Have

  • (Optional) Write/update tests under /tests for feat and test PR.
  • (Optional) Write/update docs under /docs for docs and ci PR.

@ntudy ntudy changed the title better readme feat(ux): better readme Aug 24, 2025
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

@ntudy ntudy requested a review from Copilot August 25, 2025 02:19
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

@ntudy ntudy requested a review from Copilot August 25, 2025 02:32
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

@shiqian-su shiqian-su merged commit e226ad4 into main Aug 25, 2025
8 checks passed
@shiqian-su shiqian-su deleted the dev-better-readme branch August 25, 2025 03:50
Zhudongsheng75 pushed a commit to open-compass/MiroFlow that referenced this pull request Dec 27, 2025
* better readme

* add diagrams

* update default confit

---------

Co-authored-by: Yue Deng <yue.deng@miromind.ai>
BinWang28 added a commit that referenced this pull request Mar 11, 2026
fix(utils): prevent orphan processes with PR_SET_PDEATHSIG
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.

4 participants