Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Improve project testing and automations #55

Open
3 of 8 tasks
solvaholic opened this issue May 4, 2021 · 1 comment
Open
3 of 8 tasks

Improve project testing and automations #55

solvaholic opened this issue May 4, 2021 · 1 comment
Labels
bug Something isn't working

Comments

@solvaholic
Copy link
Owner

solvaholic commented May 4, 2021

Description

Running on the runner instead of in a container (see also #50 (comment), #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 ☝️ locally.

Checklist

  • Lint faster: Are shellcheck and yamllint all we need?
  • (Repo major version tag does not automatically update #33) Automate syncing major and minor version tags when a release is published or unpublished
  • Document manual procedures, including rollbacks
  • Ensure automated actions are resilient and can't break much when they run amok
  • How much of octodns-sync can be tested without creds and a DNS provider?
  • Add validation testing on .. all pushes? At least on pushes to PRs and to main
  • Organize workflow triggers and required status checks
  • Gravy: Report performance of test runs, and correlate changes with changes
@solvaholic solvaholic added the bug Something isn't working label May 4, 2021
@solvaholic solvaholic self-assigned this May 9, 2021
solvaholic added a commit that referenced this issue May 9, 2021
which removes the dependency on github/super-linter and should make
linter runs much faster

#55
@solvaholic
Copy link
Owner Author

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.

@solvaholic solvaholic removed their assignment May 10, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

1 participant