Hi Collin,

> In main.py a call to mkdtemp() was the use of the 'tempfile' module
> prefix.

Oh, that would have led to a runtime error, right?

Can you please add, to the first line of the ChangeLog entry, a note
that tells us when the regression was introduced? Such as
   (regression 2024-xx-xy)
or
   (regr. yesterday)

The motivation for such an annotation is to get us thinking about
  - how the regression could have been avoided,
  - how the regression could have been noticed earlier,
  - whether we need extra measures when we see, say, 5 regressions in a
    single week.

In this case, maybe some Python style checker would have produced a
warning, right? Would it be possible to add a Makefile rule that checks
for the absense of such warnings, and then use this Makefile rule in the
continuous integration that runs at least once a week?

> I put back the 'glpy' prefix.

Good.

Bruno




Reply via email to