Closed kylechard closed 3 years ago
@mturilli: I would like to land this by tomorrow if possible. Could you take a quick look at the documentation policies and make sure you feel comfortable with that? Otherwise, suggest changes? Thanks.
Thanks @mturilli. Before merging this can we make "Conda" optional? @SteVwonder took a look at that and relocatable binaries and such will make "Conda" packaging for a project like Flux require potentially good amounts of efforts. We want to make it a low priority compared to Spack.
The only thing that needs would be add "optional" to Conda. After that, we can merge it in.
Great. I believe in this case, it makes sense to keep the intermediate commits as people may want to know how our prose evolved. I will put MWP to merge it.
I forgot to bring this up in the meeting today. Looks like the commit message validator explicitly forbids merge commits, while our mergify config and policies allow them in exceptional circumstances. Not sure why mergify inserted a merge commit here. Any objection to me/someone manually merging?
Hmmm. No objection from me.
@SteVwonder: please go ahead and manually merge this in.
Draft of community policies from discussion in meetings.