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

Be compatible with python 2 #14

Merged
merged 9 commits into from Jan 20, 2017
Merged

Be compatible with python 2 #14

merged 9 commits into from Jan 20, 2017

Conversation

Xuanwo
Copy link
Contributor

@Xuanwo Xuanwo commented Jan 18, 2017

No description provided.

@@ -21,4 +21,5 @@
'PyYAML >= 3.1',
'qingstor-sdk',
'docutils >= 0.10',
'tqdm',
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please specify a version of the 3rd-party package (tqdm and qingstor-sdk).

print(
"Python2 on Windows do not support this key_name: %s",
key_name
)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

print(
    "WARN: Python 2 on Windows does not support this object key: %s",
    key_name
)

@@ -0,0 +1,5 @@
qingstor-sdk
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please specify a version.

def step_impl(context):
command = qsctl(
"cp",
"tmp/{filename}".format(filename="中文测试"),
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Test data and expected behavior should be stored in feature files, such as object keys, file size, success status and error code.

Signed-off-by: Xuanwo <xuanwo@yunify.com>
Signed-off-by: Xuanwo <xuanwo@yunify.com>
Signed-off-by: Xuanwo <xuanwo@yunify.com>
Signed-off-by: Xuanwo <xuanwo@yunify.com>
Signed-off-by: Xuanwo <xuanwo@yunify.com>
Signed-off-by: Xuanwo <xuanwo@yunify.com>
Signed-off-by: Xuanwo <xuanwo@yunify.com>
@Xuanwo Xuanwo force-pushed the compatible branch 4 times, most recently from 271f59e to 75cce15 Compare January 20, 2017 11:01
Signed-off-by: Xuanwo <xuanwo@yunify.com>
)
)
command.wait()

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

一般来说是空1行, flake8可能会报warning

Signed-off-by: Xuanwo <xuanwo@yunify.com>
@Xuanwo Xuanwo merged commit 966bf89 into master Jan 20, 2017
@Xuanwo Xuanwo deleted the compatible branch January 20, 2017 14:38
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.

None yet

3 participants