Skip to content

ci(workflows): organize ci matrixes - #118

Merged
lukas-frey merged 6 commits into
mainfrom
feature/106-split-up-ci-tasks-into-more-matrix-tasks
Nov 6, 2026
Merged

lukas-frey merged 6 commits into
mainfrom
feature/106-split-up-ci-tasks-into-more-matrix-tasks

Conversation

@onairmarc

Copy link
Copy Markdown
Member

Fixes #106

This pull request significantly refactors the ci-build.yml GitHub Actions workflow to simplify and modularize the CI/CD process. The main changes involve removing in-line job definitions for platform-specific builds and replacing them with reusable workflow calls, as well as cleaning up comments and improving maintainability.

Workflow modularization and simplification:

  • Replaces the in-line macOS, Windows, and Linux build and packaging jobs with calls to separate reusable workflows (ci-macos.yml, ci-windows.yml, ci-linux.yml), passing required inputs and secrets. This reduces duplication and centralizes platform-specific logic.
  • Removes detailed in-line steps for platform-specific builds (such as signing, environment setup, and artifact upload) from ci-build.yml, delegating these responsibilities to the respective platform workflows.

Comment and documentation cleanup:

  • Removes or shortens many explanatory comments throughout the workflow, focusing the file on actionable configuration and delegating detailed documentation to workflow files or external docs. [1] [2] [3] [4]

Minor improvements and consistency:

  • Updates some descriptions for clarity (e.g., version stamping description).
  • Adjusts matrix job formatting for readability and consistency across jobs.

These changes make the workflow more maintainable, easier to read, and simpler to update as platform-specific build logic changes.

@onairmarc
onairmarc requested a review from lukas-frey November 6, 2026 20:52
@onairmarc onairmarc linked an issue Nov 6, 2026 that may be closed by this pull request
@onairmarc
onairmarc force-pushed the feature/106-split-up-ci-tasks-into-more-matrix-tasks branch from 4d33974 to 98f080f Compare November 6, 2026 21:17
@onairmarc
onairmarc force-pushed the feature/106-split-up-ci-tasks-into-more-matrix-tasks branch from 98f080f to 7e2876c Compare November 6, 2026 21:19
@onairmarc

Copy link
Copy Markdown
Member Author

@copilot resolve the merge conflicts in this pull request

Copilot AI and others added 2 commits November 6, 2026 21:52
…ci-tasks-into-more-matrix-tasks

# Conflicts:
#	internal/web/share.go

Co-authored-by: onairmarc <50760632+onairmarc@users.noreply.github.com>
Co-authored-by: onairmarc <50760632+onairmarc@users.noreply.github.com>

Copilot AI commented Nov 6, 2026

Copy link
Copy Markdown
Contributor

@copilot resolve the merge conflicts in this pull request

Resolved and pushed the merge with main in commit 2a7b0fa.

Co-authored-by: onairmarc <50760632+onairmarc@users.noreply.github.com>
@lukas-frey
lukas-frey merged commit 5e249c7 into main Nov 6, 2026
26 checks passed
@lukas-frey
lukas-frey deleted the feature/106-split-up-ci-tasks-into-more-matrix-tasks branch November 6, 2026 22:23
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.

Split up CI tasks into more matrix tasks

3 participants