Improved error message when prepping only mats with missing image paths.

Moo-Ack-Productions / MCprep

Blender python addon to increase workflow for creating minecraft renders and animations

https://theduckcow.com/MCprep

GNU General Public License v3.0

287 stars 24 forks source link

Improved error message when prepping only mats with missing image paths. #611

Closed TheDuckCow closed 3 months ago

TheDuckCow commented 3 months ago

Also resolves some pep8 warnings.

Local tests pass, but let's see about the github action runner. Tested off of a version of dev initially where the find missing function wasn't yet working, to prove the new message displays then. Also cleans up error code responses (though hm maybe we should just use the MCprep error class? though it's helpful to have the specific errno codes to check against right after).

TheDuckCow commented 3 months ago

Yup that's fair, I can take a quick stab at it as I'm wrapping up. This is the last change before go-live anyhow.

TheDuckCow commented 3 months ago

It was not, in fact, as simple as I had hoped - in that it really needed updating and trickle through handling of several functions, and felt weird to just do it halfway. So I created https://github.com/Moo-Ack-Productions/MCprep/issues/612 to capture what I did so far and as a place for (anyone) completing the work. Merging this PR in the meantime.