Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 98.54%. Comparing base (859b6ef) to head (6fba1e4).
Report is 3 commits behind head on master.

@@           Coverage Diff           @@
##           master   #16042   +/-   ##
=======================================
  Coverage   98.54%   98.54%           
=======================================
  Files         169      169           
  Lines       21864    21864           
=======================================
  Hits        21545    21545           
  Misses        319      319           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Code size report:

   bare-arm:    +0 +0.000% 
minimal x86:    +0 +0.000% 
   unix x64:    +0 +0.000% standard
      stm32:    +0 +0.000% PYBV10
     mimxrt:    +0 +0.000% TEENSY40
        rp2:    +0 +0.000% RPI_PICO_W
       samd:    +0 +0.000% ADAFRUIT_ITSYBITSY_M4_EXPRESS
  qemu rv32:    +0 +0.000% VIRT_RV32

@vshymanskyy any comments on the above review?

Can you please also split out the change to makeqstrdata.py to a separate commit (or, I can do that during merge, but it's easier if you do it).

@projectgus I believe @vshymanskyy said on Discord that he is rather preoccupied at the moment (the situation in Ukraine remains difficult). So I am sure he would appreciate it if anyone can do the small things to get this unblocked and merged :)

I've done an update according to the review. I'd not rebased, so changes can be viewed via https://github.com/micropython/micropython/compare/0289e32a9afdb87d82cde42996f5bf924a17517e..efbd3f7

However I see that there's a merge conflict, so I'll rebase now.

Now rebased. Use the compare link above to see the pre-rebase changes.

This looks good now, thanks @projectgus for updating.