Replies: 3 comments 15 replies
-
Happy to improve documentation (just adding to COONTRIBUTING.md would be easiest) but for me, as mentioned, it's an official Python PEP convention, and so there is no reason to stop using it. |
Beta Was this translation helpful? Give feedback.
-
Sorry for the slow response - was off the last couple of days. I don't have strong opinions about the convention itself, but at this point we've gotten enough pushback from potential contributors that I feel we are unnecessarily choosing to die on this hill. Since these two practices have no functional difference, it doesn't seem like something worth spending so much time debating. In this case, I think we are feeling tension with others in the conda-forge community because we are choosing to draw a line in the sand about something that feels superficial. This makes us come across as inflexible, regardless of whether the policy is the "right one" or not. If we were a static team then I'd say we just pick a convention and go with it, but since we wish to invite contributions from others in the community, I think it is important to follow practices that make it easier for others to contribute. In this case, I think we should distinguish between:
I think that the burden of proof should be showing why a practice must be required across all of our repositories, rather than assuming that all of our conventions are required. We will pay a much higher social penalty by blocking things, and this will make it harder for us to build a healthy community dynamic. So in sum, I suggest that we:
That's my 2 cents - am happy to discuss this and iterate further. I think the broader topic is important to think about as the community grows. |
Beta Was this translation helpful? Give feedback.
-
I think it would be great to move some of this thread to the discussion around Heading back to the original issue I have put together a preview of what it would look like if we used mmcky/myst-nb-feedstock@63e28d7 The Pro:
Con:
|
Beta Was this translation helpful? Give feedback.
-
Opening this as a general discussion for the
EBP
community given this feedback fromconda-forge
team.conda-forge/myst-nb-feedstock#4
It appears the
conda-forge
recommendation is to specifyversion
restrictions using>
,<
, and=
operators.We use
~
for specifying a single release within amajor
release family.It is hard to argue we shouldn't use
~=
given it is in PEP 440 but I kind of sympathise with the consistency that>
and<
delivers when reviewing the requirements. It also requires no learning. I admit I was confused what~=
meant when I first saw it.I personally don't feel strongly about this issue -- but perhaps if we use
~=
we should add a comment at the top of everysetup.cfg
file that defines it?Beta Was this translation helpful? Give feedback.
All reactions