-
Notifications
You must be signed in to change notification settings - Fork 5.6k
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
[CodeStyle] format some dy2static unittests #46463
[CodeStyle] format some dy2static unittests #46463
Conversation
你的PR提交成功,感谢你对开源项目的贡献! |
@@ -153,7 +154,7 @@ def set_static_lineno(self): | |||
def set_dygraph_info(self): | |||
self.line_num = 5 | |||
self.line_index_list = [0, 1, 2, 3, 4] | |||
self.dy_rel_lineno_list = [0, 1, 2, 4, 5] | |||
self.dy_rel_lineno_list = [0, 2, 3, 5, 6] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
这里是与 node 开始处的相对行号(不含空行),因为 30 行处的 nested_func
格式化后中间多了一行(行号 1),所以从 [0, 1, 2, 4, 5]
变成了 [0, 2, 3, 5, 6]
(行号 1 之后都 +1)
请更新下冲突 |
这个PR不能用test=document_fix来跳过,需要跑一下单测 |
啊抱歉整错了 😂,没注意这个是改单测的 PR,已经重新触发全部 CI |
PR types
Others
PR changes
Others
Describe
格式化两个被禁用格式化的动转静单测 test_error 和 test_origin_info,移除 future import
更新单测行号