solvaholic / octodns-sync

GitHub Action to test and deploy DNS settings with OctoDNS
MIT License
28 stars 14 forks source link

Improve project testing and automations #55

Open solvaholic opened 3 years ago

solvaholic commented 3 years ago

Description

Running on the runner instead of in a container (see also https://github.com/solvaholic/octodns-sync/issues/50#issuecomment-830710533, #53) drastically changes what a "release" is for this project, as well as what tests and automations are required.

Expected Behavior

Meaningful tests run for any changes pushed to this repository; Where appropriate these serve as required status checks in pull requests.

The most recent numbered release can be specified in workflows by various labels. For example: @2.2.0, ~@2.2,~ and @2 are synonymous when v2.2.0 is the most recent release.

The linter and other frequently-run tests are fast.

Actual Behavior

Currently (04 May 2021) the functions of this action are not tested automatically, the release workflow is wrong, and the linter is slow af.

Possible Fix

Automate integration and validation testing; Implement as required status checks in pull requests.

Automate release tagging.

Find a faster way to run super-linter, or run individual linters on the runner.

Update Makefile to run those :point_up: locally.

Checklist

solvaholic commented 3 years ago

Sweet, the short tags updated when I released v2.2.0. I'll come back to this list later on, want to sort #51 and #57 next.