From c9cc8cfd74fdb59d5450939e1772dea35d369b04 Mon Sep 17 00:00:00 2001 From: Jay Berkenbilt Date: Sun, 31 Jul 2022 08:19:03 -0400 Subject: Remove lgtm banner Over time, qpdf's grade has dropped on lgtm, but they are not transparent about how grades are assigned. Fixing current alerts, in my opinion, reduces clarity and maintainability of the code in the name of performance in non-critical sections of code. Some analysis by m-holger suggests that fixing some of the current alerts actually degrades performance (slightly) while fixing others results in insignificant improvements. The quality of qpdf can be measured in other ways, such as its extensive test suite, documentation, and long track record of reliably manipulating PDFs with high performance, few bugs, and few external dependencies. The lgtm rating is a distraction at best. --- README-maintainer | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) (limited to 'README-maintainer') diff --git a/README-maintainer b/README-maintainer index 29291634..c2967c99 100644 --- a/README-maintainer +++ b/README-maintainer @@ -291,7 +291,8 @@ RELEASE PREPARATION * Check for open fuzz crashes at https://oss-fuzz.com -* Check lgtm: https://lgtm.com/projects/g/qpdf/qpdf/?mode=list +* Check lgtm: https://lgtm.com/projects/g/qpdf/qpdf/?mode=list for + anything worth fixing * Check all open issues and pull requests in github and the sourceforge trackers. See ~/scripts/github-issues. Don't forget pull -- cgit v1.2.3-54-g00ecf