langgenius/dify-official-plugins
1
0
Fork 0
mirror of https://github.com/langgenius/dify-official-plugins.git synced 2026-07-21 17:45:23 -04:00
Code Issues 54 Packages Projects Releases Wiki Activity

[PR #602] [MERGED] Docs update pull request template to include version control checklist #1455

New Issue
Closed
opened 2026-02-16 10:22:59 -05:00 by yindo · 0 comments
yindo commented 2026-02-16 10:22:59 -05:00
Owner
Copy Link
Copy Source

📋 Pull Request Information

Original PR: https://github.com/langgenius/dify-official-plugins/pull/602
Author: @fdb02983rhy
Created: 3/28/2025
Status: ✅ Merged
Merged: 3/30/2025
Merged by: @crazywoola

Base: main ← Head: docs-update-pull-request-template-to-include-version-control-checklist

📝 Commits (10+)

  • b71bde9 chore: update pull request template to include version control checklist
  • d3518c5 docs: enhance pull request template with detailed versioning guidelines
  • b70282e docs: refine pull request template for clarity and consistency in version control and testing sections
  • de136f3 Update pull_request_template.md
  • d8921bc Update pull_request_template.md
  • c8c8a06 docs: update pull request template to improve clarity on evidence requirements and testing guidelines
  • a9ae02e Update pull_request_template.md
  • a510c50 delete cross folder changes to pass GHA
  • 1253c34 Revert "delete cross folder changes to pass GHA"
  • 7ea4507 Update pull_request_template.md

📊 Changes

6 files changed (+63 additions, -38 deletions)

View changed files

➕ .assets/test-examples/llm-plugin-tests/llm_test_example.md (+9 -0)
➕ .assets/test-examples/llm-plugin-tests/llm_test_example_1.png (+0 -0)
➕ .assets/test-examples/llm-plugin-tests/llm_test_example_2.png (+0 -0)
➕ .assets/test-examples/llm-plugin-tests/llm_test_example_3.png (+0 -0)
➕ .assets/test-examples/llm-plugin-tests/llm_test_example_4.png (+0 -0)
📝 .github/pull_request_template.md (+54 -38)

📄 Description

Related Issue or Context

Update pull request template to include version control checklist, environment and LLM models test examples

Type of Change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that may cause existing functionality to not work as expected)
  • Documentation update
  • Code refactoring
  • Other

Test Evidence

Important

Visual proof is required for Bug fixes, New features, Breaking changes:

  • Screenshots or Video/GIF (if applicable):

Note

For Non-LLM Plugin Changes:

  • Bug fixes:
    • Show the fix working
  • New features:
    • Demonstrate the functionality
  • Breaking changes:
    • Show both old and new behavior

For LLM Plugin Changes:

  • Bug fixes:
    • Show the fix working with example inputs/outputs
  • New features:
    • Demonstrate the functionality with example inputs/outputs
  • Breaking changes (requires comprehensive testing):
    • Conversation & Interaction:
      • Conversation sequence correctness:
        • System message handling
        • Proper turn-taking (user→assistant messages)
      • Tool usage demonstrations (if applicable):
        • Multi-round tool interactions
        • Appropriate handling of tool outputs
    • Input/Output Handling:
      • Multimodal input handling (images, PDFs, audio, video if applicable)
      • Multimodal output generation (images, PDFs, audio, video if applicable)
      • Structured output format (if applicable)
    • Metrics:
      • Token consumption metrics
    • Others:
      • Reasoning process (if applicable, e.g. tool use)

Environment Verification

Important

Please confirm your testing environment:

  • Changes tested in a clean/isolated environment
  • Test environment matches production configuration
  • No cached data influenced the test results

🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/langgenius/dify-official-plugins/pull/602 **Author:** [@fdb02983rhy](https://github.com/fdb02983rhy) **Created:** 3/28/2025 **Status:** ✅ Merged **Merged:** 3/30/2025 **Merged by:** [@crazywoola](https://github.com/crazywoola) **Base:** `main` ← **Head:** `docs-update-pull-request-template-to-include-version-control-checklist` --- ### 📝 Commits (10+) - [`b71bde9`](https://github.com/langgenius/dify-official-plugins/commit/b71bde979ff248e3c4010cc36bbcab3de69c8a3e) chore: update pull request template to include version control checklist - [`d3518c5`](https://github.com/langgenius/dify-official-plugins/commit/d3518c5abc807916e915aa2d7940498370df3452) docs: enhance pull request template with detailed versioning guidelines - [`b70282e`](https://github.com/langgenius/dify-official-plugins/commit/b70282ec0b017a3fed5561f66034f0508994c7f7) docs: refine pull request template for clarity and consistency in version control and testing sections - [`de136f3`](https://github.com/langgenius/dify-official-plugins/commit/de136f373442d0e75b6f132bc7bc5f3cc366dc49) Update pull_request_template.md - [`d8921bc`](https://github.com/langgenius/dify-official-plugins/commit/d8921bcae040a23dff8b3939fa0f39c984d41d89) Update pull_request_template.md - [`c8c8a06`](https://github.com/langgenius/dify-official-plugins/commit/c8c8a06d1e4b5021d5cc70c7f0ca50fb34953c1a) docs: update pull request template to improve clarity on evidence requirements and testing guidelines - [`a9ae02e`](https://github.com/langgenius/dify-official-plugins/commit/a9ae02e844471b2f707cfcd061f5804b7efa95f1) Update pull_request_template.md - [`a510c50`](https://github.com/langgenius/dify-official-plugins/commit/a510c5078fdf7bb22fb2ff2362de4b93d2ac0083) delete cross folder changes to pass GHA - [`1253c34`](https://github.com/langgenius/dify-official-plugins/commit/1253c34a152f7fbdfc09a9cbc449e370f5a43679) Revert "delete cross folder changes to pass GHA" - [`7ea4507`](https://github.com/langgenius/dify-official-plugins/commit/7ea4507aa773c713f1068ebac0386833f822dddf) Update pull_request_template.md ### 📊 Changes **6 files changed** (+63 additions, -38 deletions) <details> <summary>View changed files</summary> ➕ `.assets/test-examples/llm-plugin-tests/llm_test_example.md` (+9 -0) ➕ `.assets/test-examples/llm-plugin-tests/llm_test_example_1.png` (+0 -0) ➕ `.assets/test-examples/llm-plugin-tests/llm_test_example_2.png` (+0 -0) ➕ `.assets/test-examples/llm-plugin-tests/llm_test_example_3.png` (+0 -0) ➕ `.assets/test-examples/llm-plugin-tests/llm_test_example_4.png` (+0 -0) 📝 `.github/pull_request_template.md` (+54 -38) </details> ### 📄 Description ## Related Issue or Context Update pull request template to include version control checklist, environment and LLM models test examples ## Type of Change <!-- Put an `x` in all the boxes that apply --> - [ ] Bug fix (non-breaking change which fixes an issue) - [ ] New feature (non-breaking change which adds functionality) - [ ] Breaking change (fix or feature that may cause existing functionality to not work as expected) - [x] Documentation update - [ ] Code refactoring - [ ] Other ## Test Evidence > [!IMPORTANT] > Visual proof is required for Bug fixes, New features, Breaking changes: - Screenshots or Video/GIF (if applicable): > [!NOTE] > For Non-LLM Plugin Changes: > - **Bug fixes**: > - [ ] Show the fix working > - **New features**: > - [ ] Demonstrate the functionality > - **Breaking changes**: > - [ ] Show both old and new behavior > > For LLM Plugin Changes: > - **Bug fixes**: > - [ ] Show the fix working with example inputs/outputs > - **New features**: > - [ ] Demonstrate the functionality with example inputs/outputs > - **Breaking changes** (requires comprehensive testing): > - **Conversation & Interaction**: > - Conversation sequence correctness: > - [ ] System message handling > - [ ] Proper turn-taking (user→assistant messages) > - Tool usage demonstrations (if applicable): > - [ ] Multi-round tool interactions > - [ ] Appropriate handling of tool outputs > - **Input/Output Handling**: > - [ ] Multimodal input handling (images, PDFs, audio, video if applicable) > - [ ] Multimodal output generation (images, PDFs, audio, video if applicable) > - [ ] Structured output format (if applicable) > - **Metrics**: > - [ ] Token consumption metrics > - **Others**: > - [ ] Reasoning process (if applicable, e.g. tool use) ### Environment Verification > [!IMPORTANT] > Please confirm your testing environment: - [ ] Changes tested in a clean/isolated environment - [ ] Test environment matches production configuration - [ ] No cached data influenced the test results --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
yindo added the pull-request label 2026-02-16 10:22:59 -05:00
yindo closed this issue 2026-02-16 10:22:59 -05:00
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
bug
bug
bug
dependencies
documentation
enhancement
good first issue
invalid
pull-request
Mirrored from GitHub Pull Request
 question
stale
No Label pull-request
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
yindo
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: langgenius/dify-official-plugins#1455
Delete Branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?