[PR #142] [MERGED] fix: update DifyPluginEnv to set default for REMOTE_INSTALL_URL #177

Closed
opened 2026-02-15 21:16:05 -05:00 by yindo · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/langgenius/dify-plugin-sdks/pull/142
Author: @Yeuoly
Created: 5/14/2025
Status: Merged
Merged: 5/14/2025
Merged by: @Yeuoly

Base: mainHead: fix/local-launch-failed


📝 Commits (4)

  • f85f2d9 feat: update DifyPluginEnv to set default for REMOTE_INSTALL_URL and add unit tests for environment configurations
  • e0348b3 fix: update condition for install_url check in Plugin class
  • b04c8bd Merge branch 'main' into fix/local-launch-failed
  • 7698306 apply ruff

📊 Changes

1 file changed (+51 additions, -0 deletions)

View changed files

python/tests/config/test_config.py (+51 -0)

📄 Description

add unit tests for environment configurations

  • Set default value of REMOTE_INSTALL_URL to None in DifyPluginEnv.
  • Introduced unit tests for various installation methods including local, remote, and serverless configurations, ensuring proper functionality without parameters and with specific settings.

🔄 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-plugin-sdks/pull/142 **Author:** [@Yeuoly](https://github.com/Yeuoly) **Created:** 5/14/2025 **Status:** ✅ Merged **Merged:** 5/14/2025 **Merged by:** [@Yeuoly](https://github.com/Yeuoly) **Base:** `main` ← **Head:** `fix/local-launch-failed` --- ### 📝 Commits (4) - [`f85f2d9`](https://github.com/langgenius/dify-plugin-sdks/commit/f85f2d9310ba8b098c05d785fee21408fcb47d6c) feat: update DifyPluginEnv to set default for REMOTE_INSTALL_URL and add unit tests for environment configurations - [`e0348b3`](https://github.com/langgenius/dify-plugin-sdks/commit/e0348b399dd9d669f844dce51361d2bf5e205664) fix: update condition for install_url check in Plugin class - [`b04c8bd`](https://github.com/langgenius/dify-plugin-sdks/commit/b04c8bd207276b3adf6f24022ccf81814f8c1c88) Merge branch 'main' into fix/local-launch-failed - [`7698306`](https://github.com/langgenius/dify-plugin-sdks/commit/769830600b84ef49f88fd257101cf98bdbce6ca8) apply ruff ### 📊 Changes **1 file changed** (+51 additions, -0 deletions) <details> <summary>View changed files</summary> ➕ `python/tests/config/test_config.py` (+51 -0) </details> ### 📄 Description add unit tests for environment configurations - Set default value of REMOTE_INSTALL_URL to None in DifyPluginEnv. - Introduced unit tests for various installation methods including local, remote, and serverless configurations, ensuring proper functionality without parameters and with specific settings. --- <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-15 21:16:05 -05:00
yindo closed this issue 2026-02-15 21:16:05 -05:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: langgenius/dify-plugin-sdks#177