Skip to content

Conversation

@cakebaker
Copy link
Contributor

@cakebaker cakebaker commented Oct 6, 2025

This PR is an attempt to make the CICD.yml workflow work again. It currently fails with the following error and none of its checks are executed.

[Invalid workflow file: .github/workflows/CICD.yml#L1](https://github.com/uutils/coreutils/actions/runs/18274741656/workflow)
(Line: 476, Col: 12): A sequence was not expected

@cakebaker cakebaker marked this pull request as draft October 6, 2025 09:10
@cakebaker cakebaker changed the title ci: add missing "shell: bash" to CICD workflow ci: fix broken CICD workflow Oct 6, 2025
@cakebaker cakebaker force-pushed the ci_fix_cicd_workflow branch from 7e9966a to 98f617a Compare October 6, 2025 09:31
@cakebaker cakebaker marked this pull request as ready for review October 6, 2025 09:35
@cakebaker cakebaker requested a review from sylvestre October 6, 2025 09:42
@github-actions
Copy link

github-actions bot commented Oct 6, 2025

GNU testsuite comparison:

Skip an intermittent issue tests/tail/overlay-headers (fails in this run but passes in the 'main' branch)

@sylvestre sylvestre merged commit 1b74e7d into uutils:main Oct 6, 2025
98 checks passed
@codspeed-hq
Copy link

codspeed-hq bot commented Oct 6, 2025

CodSpeed Performance Report

Merging #8831 will not alter performance

Comparing cakebaker:ci_fix_cicd_workflow (98f617a) with main (5e076fe)1

Summary

✅ 79 untouched
⏩ 73 skipped2

Footnotes

  1. No successful run was found on main (98f617a) during the generation of this report, so 5e076fe was used instead as the comparison base. There might be some changes unrelated to this pull request in this report.

  2. 73 benchmarks were skipped, so the baseline results were used instead. If they were deleted from the codebase, click here and archive them to remove them from the performance reports.

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