Skip to content

change console output format from plan to colored in gradle build of ci#3364

Merged
sfeilmeier merged 1 commit intoOpenEMS:developfrom
girasolenergy:feature/colored
Oct 20, 2025
Merged

change console output format from plan to colored in gradle build of ci#3364
sfeilmeier merged 1 commit intoOpenEMS:developfrom
girasolenergy:feature/colored

Conversation

@miettal
Copy link
Contributor

@miettal miettal commented Oct 15, 2025

from gradle 9.1, it support colord option (I contributed this). --console=rich break ci output, because it designed rich terminal(VT100). but this option is for ci, not break ci output.

from gradle 9.1, it support colord option (I contributed this).
--console=rich break ci output, because it designed rich
terminal(VT100). but this option is for ci, not break ci output.
@codecov
Copy link

codecov bot commented Oct 15, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.

Additional details and impacted files
@@              Coverage Diff              @@
##             develop    #3364      +/-   ##
=============================================
+ Coverage      59.78%   59.82%   +0.04%     
  Complexity       112      112              
=============================================
  Files           2785     2785              
  Lines         120785   120785              
  Branches        9015     9015              
=============================================
+ Hits           72205    72249      +44     
+ Misses         45901    45846      -55     
- Partials        2679     2690      +11     
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@miettal
Copy link
Contributor Author

miettal commented Oct 15, 2025

Oh!!

This project output is hooked! this flag is not needed!!

https://github.com/OpenEMS/openems/blame/799ee37a0a380da5867da71ceb91df5e4647a378/build.gradle#L55

@sfeilmeier sfeilmeier requested a review from da-Kai October 20, 2025 08:19
@da-Kai da-Kai requested a review from sfeilmeier October 20, 2025 09:14
@sfeilmeier sfeilmeier merged commit c8e40fc into OpenEMS:develop Oct 20, 2025
7 checks passed
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.

3 participants