Skip to content
This repository has been archived by the owner on Mar 11, 2023. It is now read-only.

Add default values ? #77

Closed
Seluj78 opened this issue Oct 13, 2020 · 2 comments
Closed

Add default values ? #77

Seluj78 opened this issue Oct 13, 2020 · 2 comments

Comments

@Seluj78
Copy link
Collaborator

Seluj78 commented Oct 13, 2020 •
edited
Loading

https://github.com/Seluj78/Potodo/blob/86cb4b86b22b29c5b285868b7a4487b427bd3be4/potodo/potodo.py#L67-L83
@Seluj78
Copy link
Collaborator Author

Seluj78 commented Oct 13, 2020

@JulienPalard Do you think this is a good practice ? They are set by the main function anyway but it would allow an easier testing so we don't have to set the values everytime in the test fixtures

@Seluj78
Copy link
Collaborator Author

Seluj78 commented Dec 27, 2020

The tests don't work that way now.

@Seluj78 Seluj78 closed this as completed Dec 27, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@Seluj78