WeeklyTelcon_20210511

Open MPI Weekly Telecon ---

Jeff Sq. outlined the fix to the git commit checker CI: https://github.com/open-mpi/ompi/pull/8947.
- ATM, there's only a somewhat-obscure way to re-trigger this CI test: edit the PR title or PR description (to include even just adding a space). This will re-trigger the PR.
- Jeff Sq. will be looking at this this weekend to see if there's a better way to re-trigger the CI test.

We're still waiting on Datatype issues now reported in v4.1.1
- Potentially getting access to a machine to replicate after today.

In holding pattern waiting for Datatype issue (see above).
We did merge a bunch of pending 4.1.x PRs.
If we need to go back and make a 4.1.2 with a datatype fix, we'll do the Right Git Stuff to make a minimum-distance 4.1.2.

RMs had discussion about this yesterday.
- A new point was realized in the last week: clang-tidy makes us have heavy-handed style guide. I.e., we have to make decisions about all style issues -- we can't tell clang-tidy to ignore specific rules.
- This is a deal-breaker.
- Our style guide is intentionally very light touch -- very few rules.
- It sounds like clang-tidy is simply not the tool for us.
The RMs therefore propose the following:
- Accept what has already been done / merged in OPAL and tests.
- Make sure that those changes are properly PR'ed over to v5.0.x.
- Drop the pending reformatting OMPI/OSHMEM PRs. I.e., do no further reformatting.
If someone cares, see if there is another tool out there that would allow us to do "light touch" rules.
- Linux kernel community checkpatch was discussed, but offhand, we think that their style rules will not be compatible with ours.