Enable beman-tidy (--require-all mode) into beman.indices_view#14
Enable beman-tidy (--require-all mode) into beman.indices_view#14neatudarius wants to merge 1 commit into
Conversation
Issue: bemanproject/beman-tidy#256 Enable beman-tidy (--require-all mode) into beman.indices_view
|
@neatudarius I think this is ready? all the tidy checks are passing? @wermos any thoughts? |
The pre-commit check isn't passing yet: Some of these ( I can handle the missing license IDs in the files in the repo, but I am not familiar with the process for updating the files in the |
@wermos I checked and they are fixed upstream in infra now -- so an update is probably in order. Can we fix the other ones and close this out? |
|
Yep, the license issue is fixed. The only failing check is this: I don't think this library is available on Godbolt. What is the intended solution: add this on godbolt or make it so that the lint check doesn't ask for the godbolt badge? |
Issue: bemanproject/beman-tidy#256
Enable beman-tidy (--require-all mode) into beman.indices_view
Note: Badge check should be fixed before merge.