Skip to content

Fixes #28105: Use colors in CI builds#6832

Merged
amousset merged 1 commit intoNormation:branches/rudder/9.1from
amousset:bug_28105/try_to_use_colors_in_cargo_ci_builds
Jan 9, 2026
Merged

Fixes #28105: Use colors in CI builds#6832
amousset merged 1 commit intoNormation:branches/rudder/9.1from
amousset:bug_28105/try_to_use_colors_in_cargo_ci_builds

Conversation

@amousset
Copy link
Member

@amousset amousset commented Jan 8, 2026

https://issues.rudder.io/issues/28105

Works with cargo, mvn, npm, elm-tests, api-doc and probably other tooling.

The tricky part if to find colors that match the light/dark modes while avoiding setting an hardcoded background

image image image image

@amousset amousset requested a review from VinceMacBuche January 8, 2026 21:44
@amousset amousset marked this pull request as draft January 8, 2026 21:44
@amousset
Copy link
Member Author

amousset commented Jan 8, 2026

PR updated with a new commit

@amousset amousset changed the title Fixes #28105: Try to use colors in cargo CI builds Fixes #28105: Try to use colors in CI builds Jan 8, 2026
@amousset amousset changed the title Fixes #28105: Try to use colors in CI builds Fixes #28105: Use colors in CI builds Jan 8, 2026
@amousset
Copy link
Member Author

amousset commented Jan 9, 2026

PR updated with a new commit

2 similar comments
@amousset
Copy link
Member Author

amousset commented Jan 9, 2026

PR updated with a new commit

@amousset
Copy link
Member Author

amousset commented Jan 9, 2026

PR updated with a new commit

@amousset amousset marked this pull request as ready for review January 9, 2026 08:46
@Normation-Quality-Assistant
Copy link
Contributor

This PR is not mergeable to upper versions.
Since it is "Ready for merge" you must merge it by yourself using the following command:
rudder-dev merge https://github.com/Normation/rudder/pull/6832
-- Your faithful QA
Kant merge: "Morality is not the doctrine of how we may make ourselves happy, but how we may make ourselves worthy of happiness."
(https://ci.normation.com/jenkins/job/merge-accepted-pr/112268/console)

@amousset
Copy link
Member Author

amousset commented Jan 9, 2026

OK, squash merging this PR

@amousset amousset force-pushed the bug_28105/try_to_use_colors_in_cargo_ci_builds branch from f1538f9 to cb19ef7 Compare January 9, 2026 14:56
@amousset amousset merged commit cb19ef7 into Normation:branches/rudder/9.1 Jan 9, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants