Skip to content

gh actions: make build workflow actually build#229

Merged
jaysa68 merged 4 commits intomainfrom
fix-actions-build
Apr 2, 2026
Merged

gh actions: make build workflow actually build#229
jaysa68 merged 4 commits intomainfrom
fix-actions-build

Conversation

@24apricots
Copy link
Copy Markdown
Member

--evaluator streaming would be preferable, but it results in colmena exiting as successful after 2 seconds without doing anything, even if nix-eval-jobs is installed.

`--evaluator streaming` would be preferable, but it results in colmena
exiting as successful after 2 seconds without doing anything, even if
nix-eval-jobs is installed.
fixes the remaining evaluation warnings for:
'system' has been renamed to/replaced by 'stdenv.hostPlatform.system'
@24apricots 24apricots force-pushed the fix-actions-build branch 2 times, most recently from 33d22e9 to 4229127 Compare April 2, 2026 05:31
@24apricots
Copy link
Copy Markdown
Member Author

its not actually verbosing but i think its fine

Copy link
Copy Markdown
Member

@jaysa68 jaysa68 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

APPROVED!!!

@jaysa68 jaysa68 enabled auto-merge (squash) April 2, 2026 05:50
@jaysa68 jaysa68 merged commit 280ec16 into main Apr 2, 2026
2 checks passed
@jaysa68 jaysa68 deleted the fix-actions-build branch April 2, 2026 06:03
danxliu pushed a commit that referenced this pull request Apr 2, 2026
* gh actions: make build workflow actually build

`--evaluator streaming` would be preferable, but it results in colmena
exiting as successful after 2 seconds without doing anything, even if
nix-eval-jobs is installed.

* gh actions: enable verbose builds

* fix eval warning on builds

fixes the remaining evaluation warnings for:
'system' has been renamed to/replaced by 'stdenv.hostPlatform.system'
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.

2 participants