-
Notifications
You must be signed in to change notification settings - Fork 3
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
Migrate script 1 #28
base: master
Are you sure you want to change the base?
Migrate script 1 #28
Conversation
dea1181
to
a0adbfe
Compare
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.
其他的我晚点来看,先把这堆注释放到PR的comments里面。代码内部的comment应该是针对代码本身的,可以用英文写在代码中,比如某句是干什么用的。把这些东西整理好了放在comment中给reviewer看,这样会比较友好~
import uuid | ||
import cStringIO | ||
|
||
''' |
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.
这些东西写在PR的comment里面啊,不是让你写在代码里:unamused:。
然后把内容整理清楚,不需要用对话模式~
这个脚本把老的数据库转换成了新的数据库。
新链接格式是 |
@zhzhzoo 那个 |
@JeldorPKU 呃我错了。。。 |
@zhzhzoo 几个函数加几句注释说明一下功能呗~ |
@JeldorPKU done~ |
No description provided.