Skip to content

json2csv cyclomatic complexity#1784

Merged
nfahlgren merged 3 commits intomainfrom
json2csv-complexity
Sep 24, 2025
Merged

json2csv cyclomatic complexity#1784
nfahlgren merged 3 commits intomainfrom
json2csv-complexity

Conversation

@joshqsumner
Copy link
Contributor

@joshqsumner joshqsumner commented Sep 10, 2025

Describe your changes
Using helper functions to lower cyclomatic complexity per deepsource

Type of update
This is a deepsource change.

Associated issues
None

For the reviewer
See this page for instructions on how to review the pull request.

  • PR functionality reviewed in a Jupyter Notebook
  • All tests pass
  • Test coverage remains 100%
  • Documentation tested
  • New documentation pages added to plantcv/mkdocs.yml
  • Changes to function input/output signatures added to updating.md
  • Code reviewed
  • PR approved

@deepsource-io
Copy link

deepsource-io bot commented Sep 10, 2025

Here's the code health analysis summary for commits e71983e..80826ae. View details on DeepSource ↗.

Analysis Summary

AnalyzerStatusSummaryLink
DeepSource Python LogoPython✅ Success
🎯 1 occurence resolved
View Check ↗
DeepSource Test coverage LogoTest coverage✅ SuccessView Check ↗

Code Coverage Report

MetricAggregatePython
Branch Coverage100%100%
Composite Coverage100%100%
Line Coverage100%100%
New Branch Coverage100%100%
New Composite Coverage100%100%
New Line Coverage100%, ✅ Above Threshold100%, ✅ Above Threshold

💡 If you’re a repository administrator, you can configure the quality gates from the settings.

@joshqsumner joshqsumner added ready to review cleanup Code quality updates labels Sep 10, 2025
@HaleySchuhl HaleySchuhl self-requested a review September 17, 2025 15:57
@github-project-automation github-project-automation bot moved this to Pull Requests in PlantCV4 Sep 24, 2025
@nfahlgren nfahlgren added this to the PlantCV v4.10 milestone Sep 24, 2025
@nfahlgren nfahlgren merged commit b46852b into main Sep 24, 2025
5 checks passed
@github-project-automation github-project-automation bot moved this from Pull Requests to Done in PlantCV4 Sep 24, 2025
@nfahlgren nfahlgren deleted the json2csv-complexity branch September 24, 2025 16:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cleanup Code quality updates ready to review

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

3 participants