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

支持 format 配置 #37

Closed
Songggggr opened this issue Jan 31, 2024 · 20 comments
Closed

支持 format 配置 #37

Songggggr opened this issue Jan 31, 2024 · 20 comments
Milestone

Comments

@Songggggr
Copy link

想设置 indent 和 对齐方式 🙏

@joyme123
Copy link
Owner

对齐方式是指 tab 还是 space 吗?

@Songggggr
Copy link
Author

Songggggr commented Jan 31, 2024

对齐方式是指 tab 还是 space 吗?

不是,是类似于这两个 align
image

@Songggggr
Copy link
Author

再反馈一个问题,貌似现在每次插件初始化时都会去检查 thrift-ls 的版本,github 超时,导致经常报错然后使用缓存的版本,这个报错可以去除嘛? 🙏

@joyme123
Copy link
Owner

对齐方式是指 tab 还是 space 吗?

不是,是类似于这两个 align image

Align By Assign, Align By Field 可以截图看一下分别是什么效果吗?

@joyme123
Copy link
Owner

再反馈一个问题,貌似现在每次插件初始化时都会去检查 thrift-ls 的版本,github 超时,导致经常报错然后使用缓存的版本,这个报错可以去除嘛? 🙏

感谢反馈。这个问题会解决一下

@Songggggr
Copy link
Author

Songggggr commented Jan 31, 2024

对齐方式是指 tab 还是 space 吗?

不是,是类似于这两个 align image

Align By Assign, Align By Field 可以截图看一下分别是什么效果吗?

image
image
image

就是想要配置“值”的对齐方式,= 号那个无所谓,一个配置足以

@joyme123
Copy link
Owner

对齐方式是指 tab 还是 space 吗?

不是,是类似于这两个 align image

Align By Assign, Align By Field 可以截图看一下分别是什么效果吗?

image image image

就是想要配置“值”的对齐方式,= 号那个无所谓,一个配置足以

好的。最后确认一下,目前支持的应该是第二张图片,想要通过设置来支持 1,3 两张图片的效果是吧?

@Songggggr
Copy link
Author

Songggggr commented Jan 31, 2024

对齐方式是指 tab 还是 space 吗?

不是,是类似于这两个 align image

Align By Assign, Align By Field 可以截图看一下分别是什么效果吗?

image image image
就是想要配置“值”的对齐方式,= 号那个无所谓,一个配置足以

好的。最后确认一下,目前支持的应该是第二张图片,想要通过设置来支持 1,3 两张图片的效果是吧?

是的,一个配置项切换 1/2 就行,3 那张图中 = 的逻辑可以和 2 用一个配置项一起控制

@joyme123
Copy link
Owner

joyme123 commented Jan 31, 2024

image

@Songggggr 格式化配置已支持。从 github 下载 thriftls 报错的问题,还没有想到好的解决方法。这个暂时搁置一下。

从 vscode 扩展下载 v1.0.5 预发布版本,然后让其从 github 自动更新 thriftls 到 v0.1.7 即可。
image

@Songggggr
Copy link
Author

用上了,好用,感谢!
image
第一次安装会这样,必须重新 reload 编辑器让插件重启后才开始正常下载,下载完成后,就正常了。

@Songggggr
Copy link
Author

Songggggr commented Jan 31, 2024

用上了,好用,感谢! image 第一次安装会这样,必须重新 reload 编辑器让插件重启后才开始正常下载,下载完成后,就正常了。

每次启动,有时还是会报这个,这个可以改成 log 吧,毕竟不是第一次下载的话,没必要通知这个了
image

@Songggggr
Copy link
Author

格式化还有个问题,就是尾随 “,” 不会自动添加,不过 thrift 不加 “,” 也能编译通过。这个要不要考虑添加个选项
自动补全/删除 “,” (包括最后一个 ,)

@joyme123
Copy link
Owner

joyme123 commented Feb 1, 2024

@Songggggr 好的。这几个问题我有空的时候再看看怎么解决

@joyme123
Copy link
Owner

joyme123 commented Feb 6, 2024

@Songggggr 发了 v1.0.6 版本。关于那个报错提示的问题,可以在设置里关闭或降低检查更新的频率

@Songggggr
Copy link
Author

@Songggggr 发了 v1.0.6 版本。关于那个报错提示的问题,可以在设置里关闭或降低检查更新的频率

辛苦,还有两个小问题:

  1. align=field 时,注释不会对齐,go 中注释是会对齐的
    image
  2. align=assign 时,enum 中 =1 =2 不会左对齐
    image

@Songggggr
Copy link
Author

Songggggr commented Feb 6, 2024

代码片段有个小问题,在空文件上输入
image
实际会多一个 s,也就是说没有替换掉 s,而是直接添加了
image
现在是非空文件了,下面再次重复,正常
image

感觉复现条件不是空文件,而是在一个文件中,第一次触发代码片段

同时,这种情况触发的时候,提示也是有问题的,没有找到 include 的提示,但是没有触发上面的情况下,又可以找到
image

@Songggggr
Copy link
Author

image
这种情况,中间能加个空行吗?

@Songggggr
Copy link
Author

设置添加逗号后,service 中不会自动添加

@joyme123
Copy link
Owner

joyme123 commented Feb 7, 2024

@Songggggr 好的。

有几个格式化的问题是现在设计如此,因为格式化要处理的场景比较多且复杂,所以可能后面有空会去慢慢加上。

代码补全的部分,目前做的比较粗糙,因此后续有一个优化计划。上述的一些问题会在重新实现代码补全时一起解决掉。感谢反馈

@joyme123 joyme123 transferred this issue from joyme123/thriftls-vscode Feb 21, 2024
@joyme123 joyme123 added this to the v0.2.1 milestone Feb 21, 2024
@joyme123 joyme123 modified the milestones: v0.2.1, v0.2.2 Sep 9, 2024
@joyme123
Copy link
Owner

设置添加逗号后,service 中不会自动添加

这个问题在这个 PR 中解决了。一些其他问题单独建 issue 跟踪了。这个 issue 先关闭了。新的问题可以单独建 issue 跟踪 @Songggggr
#41

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

No branches or pull requests

2 participants