Enable beman-tidy --require-all (disable readme.implements until P3610)

[mtimaN]

Superseded by #63 — see https://github.com/neatudarius/scope/tree/enable-tidy-require-all

[coveralls]

coverage: 100.0%. remained the same — mtimaN:enable-tidy-require-all into bemanproject:main

[neatudarius]

LGTM, I think we should do this.

Commented more on bemanproject/beman-tidy#260 (comment)

[JeffGarland]

In order for it to pass I added a placeholder link for the paper (beman-tidy requires a wg21 paper URL).

The paper number is assigned but has not yet been published -- the work in progress is in the repo under papers, that should be sufficient to pass the check.

[neatudarius]

In order for it to pass I added a placeholder link for the paper (beman-tidy requires a wg21 paper URL).

The paper number is assigned but has not yet been published -- the work in progress is in the repo under papers, that should be sufficient to pass the check.

Yes, please take a look at the diff

He put https://wg21.link/P3610R0 (which is the good URL, with your reserved 3610 number). He wanted to say that currently the URL gives you 404 Not Found, which is expected.

Can we proceed with this PR?

[JeffGarland]

I'd really rather not have a broken link in the site -- can we not just link back to the repo? I'm guessing that won't work bc it needs to point at wg21?

[neatudarius]

I'd really rather not have a broken link in the site -- can we not just link back to the repo? I'm guessing that won't work bc it needs to point at wg21?

It does not work as per current beman-tidy implementation (which I think it's OK).

We can then disable this check after bemanproject/beman-tidy#279 is done.

But more important, we need codeowner input on bemanproject/beman-tidy#260 (comment)

[neatudarius]

Fixed in #63, discarding current PR.