Skip to content

Declare Python dependency in SKILL.md frontmatter#4

Merged
hustcc merged 2 commits into
masterfrom
copilot/add-python-dependency-field
Jan 23, 2026
Merged

Declare Python dependency in SKILL.md frontmatter#4
hustcc merged 2 commits into
masterfrom
copilot/add-python-dependency-field

Conversation

Copilot AI commented Jan 23, 2026

Copy link
Copy Markdown
Contributor

The SKILL.md frontmatter was missing the dependency.python field to declare the requests>=2.28.0 dependency required by scripts/generate.py.

Changes

  • Added dependency.python: requests>=2.28.0 to the SKILL.md YAML frontmatter
---
name: chart-visualization
description: This skill should be used when the user wants to visualize data...
dependency:
  python: requests>=2.28.0
---

This ensures the Python dependency is properly declared for the chart generation script, which uses requests for HTTP calls to the visualization service.

Original prompt

修复一下下面的问题:

SKILL.md 前言区缺少 dependency.python 字段声明 Python 依赖。

解法:

补充依赖声明:在 SKILL.md 前言区添加 dependency.python 字段,声明 requests>=2.28.0(scripts/generate.py 使用了该库)


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Co-authored-by: hustcc <7856674+hustcc@users.noreply.github.com>
Copilot AI changed the title [WIP] Add dependency.python declaration for Python requirements Declare Python dependency in SKILL.md frontmatter Jan 23, 2026
Copilot AI requested a review from hustcc January 23, 2026 08:51
@hustcc
hustcc marked this pull request as ready for review January 23, 2026 09:09
@hustcc

hustcc commented Jan 23, 2026

Copy link
Copy Markdown
Member

/gemini review

@gemini-code-assist gemini-code-assist Bot 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.

Code Review

This pull request correctly adds the missing Python dependency requests to the SKILL.md frontmatter, which is required by scripts/generate.py. The change is accurate and addresses the issue. I have included one suggestion to enhance the dependency's version constraint for better long-term stability.

Comment thread chart-visualization/SKILL.md
@hustcc
hustcc merged commit c9f058b into master Jan 23, 2026
2 checks passed
@hustcc
hustcc deleted the copilot/add-python-dependency-field branch January 23, 2026 16:06
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