Skip to content

feat!: add output formatter #124

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 4 commits into from
Jun 18, 2025
Merged

feat!: add output formatter #124

merged 4 commits into from
Jun 18, 2025

Conversation

neilime
Copy link
Owner

@neilime neilime commented Jun 17, 2025

Separate output formatting in a dedicated domain

  • Allows to specify formatter(s) via cli argument
  • Supports following formatters:
    • plain (default original one)
    • Github Actions
    • Gitlab CI

@neilime neilime self-assigned this Jun 17, 2025
@neilime neilime force-pushed the feat/add-ouput-formatter branch 2 times, most recently from 6297e61 to d1aea24 Compare June 17, 2025 18:37
Signed-off-by: Emilien Escalle <emilien.escalle@escemi.com>
@neilime neilime force-pushed the feat/add-ouput-formatter branch from d1aea24 to f5bf042 Compare June 17, 2025 18:40
Copy link

codecov bot commented Jun 17, 2025

Codecov Report

Attention: Patch coverage is 96.95946% with 9 lines in your changes missing coverage. Please review.

Project coverage is 96.00%. Comparing base (6f1b011) to head (93f1950).
Report is 4 commits behind head on main.

Files with missing lines Patch % Lines
src/CssLint/Cli.php 94.73% 4 Missing ⚠️
src/CssLint/Output/FileOutput.php 86.66% 2 Missing ⚠️
...ssLint/Output/Formatter/GithubActionsFormatter.php 96.29% 2 Missing ⚠️
src/CssLint/Output/Formatter/GitlabCiFormatter.php 98.64% 1 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff              @@
##               main     #124      +/-   ##
============================================
+ Coverage     95.89%   96.00%   +0.10%     
- Complexity      417      489      +72     
============================================
  Files            29       36       +7     
  Lines          1292     1526     +234     
============================================
+ Hits           1239     1465     +226     
- Misses           53       61       +8     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@neilime neilime changed the title feat: add output formatter feat!: add output formatter Jun 17, 2025
Signed-off-by: Emilien Escalle <emilien.escalle@escemi.com>
@neilime neilime force-pushed the feat/add-ouput-formatter branch 2 times, most recently from a39ef5d to d780d06 Compare June 18, 2025 09:05
Signed-off-by: Emilien Escalle <emilien.escalle@escemi.com>
@neilime neilime force-pushed the feat/add-ouput-formatter branch from d780d06 to bf955e7 Compare June 18, 2025 09:12
Signed-off-by: Emilien Escalle <emilien.escalle@escemi.com>
@neilime neilime force-pushed the feat/add-ouput-formatter branch from af59fb9 to 93f1950 Compare June 18, 2025 16:37
@neilime neilime merged commit 3dbb045 into main Jun 18, 2025
6 checks passed
@neilime neilime deleted the feat/add-ouput-formatter branch June 18, 2025 16:41
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.

1 participant