The models still need to be resaved, but we need to bump the version number before doing that. 85178a5 and dffed1b only add/strip newlines if the version number is 7.1.0 or higher, so just resaving the models now will break some tests.

PR for resaved models: NetLogo/models#449

I think there's something weird going on with GitHub Actions. 68f1a2e shouldn't be able to pass the tests until NetLogo/models#449 is merged, but somehow it did. Now that it's actually merged, it's back to failing the tests again.

@kgmt0 If you look at the tests that are currently failing on wip-7.1, they don't seem to relate at all to the models repo, so I think they are unrelated. Also, I don't see why the commit you referenced should have failed the tests, because even though you didn't merge the PR in the models repo, you still pointed this PR to the correct commit in the models repo.

If you look at the tests that are currently failing on wip-7.1, they don't seem to relate at all to the models repo, so I think they are unrelated.

That's even more strange. How did the same failures not happen for 68f1a2e? The exact same commit in my fork has failing tests.

Also, I don't see why the commit you referenced should have failed the tests, because even though you didn't merge the PR in the models repo, you still pointed this PR to the correct commit in the models repo.

The commit only exists in my own fork right now. I haven't pushed anything to official models repo yet.

Hmm, that does sound wrong. GitHub has apparently been having database issues recently, so maybe that's the explanation?