Skip to content
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

Fix issue with missing CSV output values #1784

Merged
merged 5 commits into from
Apr 24, 2024

Conversation

maxschmeling
Copy link
Contributor

@maxschmeling maxschmeling commented Apr 19, 2024

Description

This fixes an issue with CSV output where values would be empty because formatData was changing the field names instead of using a formatter to set the header display.

Pull request checklist

New release checklist

Steps to Test

  1. Check out PR.
  2. Run npm run build
  3. Run something like a slowlogs command that has fields with multiple words and make sure it is showing values for those fields

Copy link
Contributor

github-actions bot commented Apr 19, 2024

Dependency Review

✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.

OpenSSF Scorecard

PackageVersionScoreDetails

Scanned Manifest Files

Copy link

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud

@maxschmeling maxschmeling merged commit c1a36a7 into trunk Apr 24, 2024
17 checks passed
@maxschmeling maxschmeling deleted the fix/BB8-11237-csv-missing-output branch April 24, 2024 14:36
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