Skip to content

CI: run tests and verification before building artifacts#30

Merged
meanmail merged 3 commits intomainfrom
check-vitalik-unanswered
Mar 26, 2026
Merged

CI: run tests and verification before building artifacts#30
meanmail merged 3 commits intomainfrom
check-vitalik-unanswered

Conversation

@meanmail
Copy link
Copy Markdown
Contributor

Summary

  • Reorder CI steps: run tests and plugin verification before building and uploading artifacts
  • Fail fast on test/verification errors without wasting time on artifact packaging

Test plan

  • Verify CI passes with the new step order
  • Confirm artifacts are still uploaded after successful tests

🤖 Generated with Claude Code

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@meanmail meanmail force-pushed the check-vitalik-unanswered branch from 1d292cf to 18c6b8f Compare March 26, 2026 09:59
meanmail and others added 2 commits March 26, 2026 14:08
- ci.yaml: tests + verification on PRs and push to main
- publish.yaml: build + publish on push to main and GitHub release

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Use workflow_run trigger so publishing only runs after CI passes.
Release events still trigger directly (CI doesn't run on releases).

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@meanmail meanmail closed this Mar 26, 2026
@meanmail meanmail reopened this Mar 26, 2026
@meanmail meanmail merged commit aa3aa50 into main Mar 26, 2026
0 of 3 checks passed
@meanmail meanmail deleted the check-vitalik-unanswered branch March 26, 2026 10:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant