Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

提议修改代码范围开始与结束标记 #117

Closed
wants to merge 1 commit into from

Conversation

etigerstudio
Copy link
Contributor

其实我们现在的机制是完全有效的,我主要纠结两个小点:

  • 现在用的这行英文括号里的提示文字(禁止修改与删除)的用法稍微有点不规范。
  • 现行的文字标记不是很醒目,看起来也不是特别正式。

PR里的格式主要参照的是PGP公钥的那种格式。为了保持内容上的一致,把禁止修改的提示信息也加在了后面的括号里。

实话说,改这种东西可以改,但也没必要改。主要是有个人喜好问题和版本兼容性问题,所以改不改都没有关系,以后某个大版本更新再改同样也成。

@shuzijun
Copy link
Owner

这个修改需要做到兼容,否则用户原来拉取的文件就无法提交。
同时这个标记范围的也正在寻找更加优雅的方式

@etigerstudio
Copy link
Contributor Author

我想了下,兼容这个标记,对插件代码是累赘,格局太小,不值得。更有效的,如你所讲,是思考更加优雅的实现方式,让用户可以自由地添加其他部分而不影响提交的代码段。这个问题需要跟更多的API打交道,不是一个简单的问题。

所以我就关掉这个PR了,有其他有效的想法我会再开issue交流。谢谢

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.

2 participants