GitHub Actions tests too often

[stevecheckoway]

The current GitHub Actions setup (which was mostly copied from somewhere else) builds and tests twice when pull requests are created. I think the behavior we want is to build/test once for each

• push to master;
• creation of a pull request; and
• push to a pull request branch other than the first.

After skimming this, it's not obvious to me if it's possible or not possible.

[flavorjones]

I think I've fixed this with the most recent commit on #167

[stevecheckoway]

Will that still build on a push to a pull request branch? In other words, when a PR is opened, it will build. But if a change is made to the PR and then pushed, will that build again?

[flavorjones]

🤔 Let me test that.

[flavorjones]

@stevecheckoway You were right, they weren't being re-triggered. BRB, going to fix Nokogiri's Actions config.

[stevecheckoway]

That's too bad. There's probably a way to work around it, but it seems like something GitHub needs to fix.

[flavorjones]

@stevecheckoway Ah, found a thing that works, see a7ca22b