Skip to content

Polising #370

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

Merged
merged 4 commits into from
Feb 1, 2020
Merged

Polising #370

merged 4 commits into from
Feb 1, 2020

Conversation

chenqimiao
Copy link
Contributor

@chenqimiao chenqimiao commented Jan 30, 2020

原文:
To do this, you need to annotations to define the name of the database table, the columns, and the SQL types to properties.
原翻译:
为此你需要一些用于定义数据库名称、数据库列以及将 SQL 类型映射到属性的注解。
建议:
为此你需要一些用于定义数据库表名称、数据库列以及映射到属性的 SQL 类型的注解。

@sjsdfg
Copy link
Collaborator

sjsdfg commented Jan 31, 2020

没懂这你这个翻译好在哪了。。这个这么看都更拗口了

the SQL types to properties. 将 SQL 类型映射到属性

@chenqimiao
Copy link
Contributor Author

@sjsdfg 打错了,原文是the sql type to map to properties.这半句的话见仁见智了。
不过数据库名称这里少写了表,感觉可以补充一下

@sjsdfg
Copy link
Collaborator

sjsdfg commented Jan 31, 2020

@sjsdfg 打错了,原文是the sql type to map to properties.这半句的话见仁见智了。
不过数据库名称这里少写了表,感觉可以补充一下

数据库表这个是遗漏了,但是这是两个问题在一个 PR 里面了。。后半句我越看越拗口。

to map to properties 这个就是这个注解的作用,你是想把这个提前作为 the sql type 的定语。

但是怎么看都没有原来的翻译更顺畅。

目的在中文里面放在后面做宾语还是感觉更舒服一些。

@chenqimiao
Copy link
Contributor Author

@sjsdfg 打错了,原文是the sql type to map to properties.这半句的话见仁见智了。
不过数据库名称这里少写了表,感觉可以补充一下

数据库表这个是遗漏了,但是这是两个问题在一个 PR 里面了。。后半句我越看越拗口。

to map to properties 这个就是这个注解的作用,你是想把这个提前作为 the sql type 的定语。

但是怎么看都没有原来的翻译更顺畅。

目的在中文里面放在后面做宾语还是感觉更舒服一些。

可,那我把PR修改一下,后半句不变。

@sjsdfg sjsdfg merged commit de0e00f into lingcoder:master Feb 1, 2020
@chenqimiao chenqimiao deleted the patch-1 branch February 1, 2020 11:28
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