Make proxySettings clearer

[Meo597]

一直看不懂“不经过底层传输方式”是啥意思
transportLayer 没必要指向 v2fly 的文档
翻了下代码

[Fangliding]

能不能不要在无关的部分再修标点符号(某种自动format?) 这会让change很乱 实际edit部分的改就改了

[yuhan6665]

修格式没问题 确实如果一个commit放实际改动 第二个commit放chores更好些

[Fangliding]

这些标点符号平时基本是随便打的 这改一下那改一下那blame里就会看到很奇怪的一个完全无关的标题修改了这个部分 点进去一看就改了个句号 还有一些平时的小规则format
也不认 比如字母数字结尾句号用点(这是合法句号 数学书里都用)
这种无关紧要的小改动越少越好 就像之前收到几个尾随逗号受不了了最后写了个scanner扫了一遍整个文档几十个尾随逗号一起鲨的
格式不一起format的问题是平时编辑也可能产生(不像尾随逗号长点眼还好说) so

[yuhan6665]

你看 有的人喜欢保持标点整洁 有人喜欢保持commit history整洁 大家互相理解下😅

[Fangliding]

主要还是 这种东西不像 json 有固定规则 英汉混排真要严谨各有各的规定 如果一个人一套互相盖那没完没了了 标点也没那么影响阅读 这看起来真的只是像顺手右键format的 但是在diff里review就得左右看红绿 这里改了 看一下 哦只是删改了两个标点 就

[yuhan6665]

BTW 顺便说句这就是我不喜欢 squash 的原因 正经作者的 commits 应该是充分考虑 恰当分开的 而R佬那样就只能看到一个巨大的 commit 一年以后你再看会觉得自己右手打自己左脸.gif
所以。。还是大家互相理解下

[Fangliding]

我倒是觉得squash merge还行 在一个有讨论的pull request下面后续贴十几个甚至更多小修小补都是很正常 直接rebase 等于把心智负担都丢回给提交pr的人了 让他把历史弄得适合进主线 我之前想维持一个干净的多commit历史很麻烦 因为可能先A再B 回头去改A定义出来的一个函数的签名 就得rebase fixup两次 先改A的定义再改B的调用 再更复杂的情况各种就得剪确保得到一个ok的历史 代码的逻辑比纯文本复杂很多
squash就是摆烂全部合一块 后面开发中贴多少个小补丁都没啥问题 也挺符合一个pr做一件事 如果pr里同时包含feature再包含一些其他模块的refine应该很多人都会要求拆开吧

[Meo597]

标点空格其实我无所谓

是 vscode 自动格式化 markdown 加空格
主要变的都是空格

我有时候看到那地方变色会顺手改一下大小写或全角标点

有需要的话我可以 pr 一个 lint 上来
自动配好 vscode 插件啥的

[yuhan6665]

标点空格其实我无所谓

是 vscode 自动格式化 markdown 加空格 主要变的都是空格

我有时候看到那地方变色会顺手改一下大小写或全角标点

有需要的话我可以 pr 一个 lint 上来 自动配好 vscode 插件啥的

是的 我个人不太注意格式细节
我的建议是 这个项目参与者众多 如果有格式要求 应该考虑弄成 CI 自动的 手工处理不 efficient

[yuhan6665]

Accepted thanks!