Skip to content
This repository was archived by the owner on Oct 12, 2023. It is now read-only.

FIx issue #62 to support float as size_kb in create_temp_file() #63

Merged
haroldrandom merged 6 commits intoAzure:masterfrom
Juliehzl:temp
Jul 23, 2020
Merged

FIx issue #62 to support float as size_kb in create_temp_file() #63
haroldrandom merged 6 commits intoAzure:masterfrom
Juliehzl:temp

Conversation

@Juliehzl
Copy link
Contributor

@Juliehzl Juliehzl commented Jul 14, 2020

fix #62

@Juliehzl Juliehzl changed the title create_temp_file FIx issue #62 to support float as size_kb in create_temp_file() Jul 14, 2020
@azure-pipelines
Copy link

Commenter does not have sufficient privileges for PR 63 in repo Azure/azure-python-devtools

@qwordy
Copy link
Member

qwordy commented Jul 21, 2020

Maybe you can add a description of size_kb to declare that it supports float number.

def create_temp_file(self, size_kb, full_random=False):

@Juliehzl
Copy link
Contributor Author

Maybe you can add a description of size_kb to declare that it supports float number.

def create_temp_file(self, size_kb, full_random=False):

sure

Copy link
Member

@qwordy qwordy left a comment

Choose a reason for hiding this comment

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

Maybe you can add a description of size_kb to declare that it supports float number.

def create_temp_file(self, size_kb, full_random=False):

@Juliehzl Juliehzl requested a review from qwordy July 21, 2020 07:50
@haroldrandom haroldrandom merged commit 107aff9 into Azure:master Jul 23, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

create_temp_file() need to create in bytes or accept float as parameter

3 participants