ci: share LuaRocks quality gate #392
No reviewers
Labels
No labels
bug
documentation
duplicate
enhancement
good first issue
help wanted
invalid
question
wontfix
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
barrettruth/cp.nvim!392
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "ci/consolidate-luarocks-workflow"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Problem
The stable and nightly LuaRocks deploy workflows both carried the same inline quality-gate polling script, making the two files look almost duplicated and increasing the chance that future fixes drift.
The two workflow files should still remain separate because they are event-specific: stable publishes on version tags, nightly publishes on main pushes/manual dispatch. Combining them would likely surface skipped or blocked jobs on the wrong event in Forgejo checks.
Solution
Move the shared quality-gate polling logic to
.forgejo/scripts/wait-for-quality.shand have both workflows call it throughnix shell nixpkgs#curl nixpkgs#jq.Keep the stable workflow rendering as
deploy / LuaRocks, and rename the nightly job toNightlyso it renders asdeploy / Nightly.Validation:
bash -n .forgejo/scripts/wait-for-quality.shshellcheck .forgejo/scripts/wait-for-quality.sh.forgejo/workflows/*.yamlwith PyYAML via Nixgit diff --check