Skip to content

Conversation

@oech3
Copy link
Contributor

@oech3 oech3 commented Jan 17, 2026

@github-actions
Copy link

GNU testsuite comparison:

GNU test failed: tests/tail/follow-name. tests/tail/follow-name is passing on 'main'. Maybe you have to rebase?

@codspeed-hq
Copy link

codspeed-hq bot commented Jan 17, 2026

CodSpeed Performance Report

Merging this PR will degrade performance by 4.8%

Comparing oech3:patch-6 (1ba7c86) with main (038a08b)

Summary

⚡ 1 improved benchmark
❌ 1 regressed benchmark
✅ 280 untouched benchmarks
⏩ 38 skipped benchmarks1

⚠️ Please fix the performance issues or acknowledge them on CodSpeed.

Performance Changes

Mode Benchmark BASE HEAD Efficiency
Memory cp_large_file[16] 114.1 KB 119.9 KB -4.8%
Memory numfmt_large_numbers_si[10000] 4.8 MB 4.6 MB +4.08%

Footnotes

  1. 38 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.

@sylvestre
Copy link
Contributor

sylvestre commented Jan 17, 2026

sorry, what changed this we closed it ? :)

@oech3
Copy link
Contributor Author

oech3 commented Jan 17, 2026

This is fix for docs.tar.zst at https://github.com/uutils/coreutils/releases/tag/latest-commit

@sylvestre
Copy link
Contributor

can you say more ? :)

@oech3
Copy link
Contributor Author

oech3 commented Jan 17, 2026

I might misunderstanding, but in my understanding, #5579 was closed by excluding [ at GNUMakefile's uudoc generation step. But I accidentaly uploaded the probrematic zsh completion for [ to release.

@oech3
Copy link
Contributor Author

oech3 commented Jan 17, 2026

This is not a joke PR. If this was included to 0.6.0, someone using zsh would report bad pattern: [.

(I did'nt notice the case because I'm on fish. Sorry.)

@sylvestre sylvestre merged commit 0333fb5 into uutils:main Jan 17, 2026
156 of 157 checks passed
@oech3 oech3 deleted the patch-6 branch January 17, 2026 19:22
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