added fixed docstirngs #739
Conversation
jakubczakon
commented
Oct 18, 2021
jakubczakon
commented
- added fixed docstrings
| If 'None' it will be parsed from the `name` argument. | ||
| visibility(str, optional): level of visibility you want your project to have. | ||
| Can be set to: | ||
| - 'pub' for public projects |
There was a problem hiding this comment.
Old value management.ProjectVisibility should work fine. Only workspace(str, optional) was required to be updated
There was a problem hiding this comment.
It also applies to other enums
There was a problem hiding this comment.
But what is recommended then?
What is simpler for the users to use?
What do we want people to use?
@Herudaio as you own the experience of API afaik what do you recommend?
For me as an ML person when I have two options:
function(... member="contributor")function(... member=neptune.management.MemberRoles.CONTRIBUTOR)
I like the first one way more.
There was a problem hiding this comment.
I would go for the first one as the suggested one i.e. member="contributor"
Primarily as this is a follow-up on the design pattern decision we already made with regard to other enums/constants e.g. connection modes.
