Skip to content

test-1#4

Open
yennanliu wants to merge 1 commit into
mainfrom
test-1
Open

test-1#4
yennanliu wants to merge 1 commit into
mainfrom
test-1

Conversation

@yennanliu
Copy link
Copy Markdown
Owner

Description

Please include a summary of the changes and which issue is fixed (if applicable).

Fixes # (issue)

Type of Change

Please delete options that are not relevant.

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Documentation update
  • New skill added
  • Plugin update

Changes Made

Checklist

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own changes
  • JSON files are valid (marketplace.json, plugin.json)
  • SKILL.md files have proper frontmatter with description
  • Version numbers are consistent across files (if applicable)
  • I have updated the CHANGELOG.md with my changes
  • Documentation has been updated (README.md, skill descriptions)
  • All skills have been tested and work as expected

Testing

Please describe how you tested your changes:

Additional Notes

Add any other context about the pull request here.


For New Skills:

  • Skill has a clear, descriptive name
  • SKILL.md includes proper frontmatter with description
  • Skill provides actionable, professional-grade analysis
  • Skill aligns with the InvestSkill marketplace theme

For Plugin Updates:

  • plugin.json version has been bumped
  • marketplace.json has been updated with new version
  • CHANGELOG.md includes the changes

@github-actions github-actions Bot added the documentation Improvements or additions to documentation label Apr 20, 2026
Copy link
Copy Markdown
Contributor

@github-actions github-actions Bot left a comment

Choose a reason for hiding this comment

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

🎉 Thanks for opening your first pull request in InvestSkill!

We're excited to review your contribution. Please ensure:

  • All CI checks are passing
  • You've filled out the PR template
  • Documentation is updated (if needed)
  • CHANGELOG.md is updated

A maintainer will review your PR shortly. Feel free to ask questions or request feedback!

Check out our Contributing Guide for more information.

Copy link
Copy Markdown

@gemini-code-assist gemini-code-assist Bot left a comment

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 introduces several unnecessary blank lines in the CI-CD-GUIDE.md file. A review comment suggests removing this excessive whitespace to adhere to standard Markdown formatting practices and improve the maintainability of the source file.

Comment thread CI-CD-GUIDE.md
Comment on lines +14 to +16



Copy link
Copy Markdown

Choose a reason for hiding this comment

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

medium

Remove unnecessary blank lines. Excessive whitespace deviates from standard Markdown formatting practices and can make the source file harder to maintain.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant