Skip to content

Conversation

@lianjin
Copy link
Contributor

@lianjin lianjin commented Oct 27, 2025


name: Pull Request | 合并请求
about: 提交新功能、修复或改进 | Submit a new feature, fix, or improvement
labels: ["enhancement"]

变更内容 | Changes

modify deploy workflow to support two env
prod -> docker swarm
dev -> docker

相关 Issue | Related Issue

KAN-148 multi env

变更类型 | Type of Change

  • 新功能 Feature
  • 修复 Bug Fix
  • 文档 Documentation
  • 样式优化 Style
  • 重构 Refactor
  • 性能优化 Performance
  • 测试 Test
  • 构建/CI Build/CI
  • 其他 Other

检查清单 | Checklist

  • 代码已通过本地测试(含主流浏览器/终端)| Code tested locally (including major browsers/terminals)
  • 相关文档已更新(如有)| Documentation updated if needed
  • 代码风格符合项目规范 | Code style follows project conventions
  • 变更不会影响现有功能 | Changes do not break existing features
  • 已自查无敏感信息泄露 | No sensitive info is leaked

变更影响 | Impact

测试说明 | Test Instructions

其他说明 | Additional Notes

@lianjin lianjin requested review from Copilot and sfxfs October 27, 2025 14:13
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR modifies the GitHub Actions deployment workflow to support multiple environments (dev and prod) with different deployment strategies. The dev environment uses standalone Docker containers, while the prod environment uses Docker Swarm for orchestration.

Key Changes:

  • Added environment selection input (env_name) with dev/prod options
  • Split deployment logic into environment-specific jobs with appropriate runner tags
  • Replaced single swarm-deploy command with environment-aware deploy command

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@lianjin lianjin force-pushed the KAN-148-CD-Multi-Env-Plan branch from 69c6f21 to 82985c4 Compare October 28, 2025 09:01
@lianjin lianjin merged commit 02db1a1 into main Oct 29, 2025
11 checks passed
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.

3 participants