-
Notifications
You must be signed in to change notification settings - Fork 1
Open
Description
问题描述
生产代码里夹杂大量调试代码:
- console.log 和 console.error 没清理(生产代码:console.log 和 console.error 没清理 #24)
- WebSocket 实现不完善(WebSocket 实现不完善:缺类型、缺心跳、缺重连 #38)
- 配置不一致(配置不一致:Tailwind 和 PostCSS 配置方式混用 #34)
- 文档缺失(文档缺失:README 太简单 #36)
- 配置混乱(配置混乱:项目同时支持 .env 和 settings.json #42)
建议方案
1. 统一日志方案
- 用环境变量控制日志级别
- 或直接用 console,但在 production 环境自动禁用
2. 完善 WebSocket
- 加类型
- 加心跳
- 加重连
3. 统一配置
// tailwind.config.js
plugins: [],
// postcss.config.mjs
export default {
plugins: {
'@tailwindcss/postcss': {},
},
}4. 完善 README
5. 统一配置方案
只用环境变量,不用 settings.json
相关 Issue(建议关闭)
- 生产代码:console.log 和 console.error 没清理(生产代码:console.log 和 console.error 没清理 #24)
- WebSocket 实现不完善:缺类型、缺心跳、缺重连(WebSocket 实现不完善:缺类型、缺心跳、缺重连 #38)
- 配置不一致:Tailwind 和 PostCSS 配置方式混用(配置不一致:Tailwind 和 PostCSS 配置方式混用 #34)
- 文档缺失:README 太简单(文档缺失:README 太简单 #36)
- 配置混乱:项目同时支持 .env 和 settings.json(配置混乱:项目同时支持 .env 和 settings.json #42)
本 Issue 由 Memo Code 生成。
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
No labels