Hi@matt256, thanks for the PR and welcome to Matplotlib! It looks like, since you created your branch, there have been some changes on themain branch within the same file you edited. So you will need to rebase and fix some conflicts, following the instructions here:
https://matplotlib.org/devdocs/devel/development_workflow.html#rebasing-on-upstream-main

For your questionhere, you will see a failure in our tests having both added and replaced an image in the same PR. To fix that, you will need to squash the commits down to one. There are instructions for that here:
https://matplotlib.org/devdocs/devel/development_workflow.html#rewriting-commit-history

Hi@matt256, it looked like your branch got into a bit of a tangle. I thought it might be difficult to guide you through fixing that so, I hope you don't mind, I just pushed a change myself. While I was there, I also dropped your last two commits, as it looked like you usedblack or a similar auto-formatter on the test file. This created a lot of changes that were not relevant to the problem you are solving so would have made it harder to review. Matplotlib has made a conscious decision to not useblack, so formatting needs to be fixed manually. Seehere for how to check the formatting at the command line.

Hi@rcomer, thanks for you help and digging me out of my hole. You are correct. I got a bit turned around, and I did use black. I'll see if I can get these issues resolved this evening with your guidance on flake8 and then go ahead and get it squashed to get the other error to go away. couple quick questions, though:

• I noted that creating the test image via a standalone script and creating it inside the test file created a file that was about 3kb different in size. Whatever the change actually was, it caused the image comparison to fail. Because I used the same script to generate the image both inside and outside the test, I assumed it was something that the decorator was doing and it wasn't a big deal, Just wanted to note it, though.
• This likely may have been due to my late-night stupor, but before I set up my linter and formatter, I was getting flake8 errors for "line too long" throughout the entire test file. Did I see that right? I see that isn't the case anymore. I thought I may have just been the first one to get to it. Am I going crazy? It may just not matter anymore

@matt256 - the default styles all changed with Matplotlib 2.0, but the tests still default to the old style so we did not need to regenerate the images. New test should usually use the up-to-date style: e.g.@image_comparison(['contour_hatching'], remove_text=True, style='mpl20')

Please update the title of this PR; it is currently auto-generated from your branch name, but it is not very clear what it is for someone looking at notifications/the list of PRs.

Closing this as#24783 is a newer version.