Skip to content

docs(model): update#2100

Merged
chickenlj merged 2 commits into
agentscope-ai:mainfrom
jujn:docs/model-update
Jul 10, 2026
Merged

docs(model): update#2100
chickenlj merged 2 commits into
agentscope-ai:mainfrom
jujn:docs/model-update

Conversation

@jujn

@jujn jujn commented Jul 10, 2026

Copy link
Copy Markdown
Contributor

Description

[Please describe the background, purpose, changes made, and how to test this PR]

Checklist

Please check the following items before code is ready to be reviewed.

  • Code has been formatted with mvn spotless:apply
  • All tests are passing (mvn test)
  • Javadoc comments are complete and follow project conventions
  • Related documentation has been updated (e.g. links, examples, etc.)
  • Code is ready for review

@jujn jujn requested review from a team and Copilot July 10, 2026 05:11

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@codecov

codecov Bot commented Jul 10, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@oss-maintainer oss-maintainer left a comment

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.

Summary

Documentation update reflecting the migration of model providers from core to extension modules. Changes are consistent across en/zh docs.

Findings

  • [Info] change-log.md — Well-structured addition of section A.3 with clear migration steps and renumbering of subsequent sections.
  • [Info] quickstart.md — The DashScope dependency example is helpful for new users.
  • [Info] model.md — Correctly clarifies environment variable naming (DASHSCOPE_API_KEY / OPENAI_API_KEY) and ModelCreationContext role.

Verdict

Accurate, consistent bilingual docs update. LGTM.


Automated review by github-manager

@chickenlj chickenlj merged commit 9a9bf2b into agentscope-ai:main Jul 10, 2026
6 checks passed
@jujn jujn deleted the docs/model-update branch July 10, 2026 09:43
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