Skip to content

Quick color check to qc#1920

Merged
nfahlgren merged 8 commits into
v5.0from
quick_color_check-to-qc
Jun 15, 2026
Merged

Quick color check to qc#1920
nfahlgren merged 8 commits into
v5.0from
quick_color_check-to-qc

Conversation

@joshqsumner

@joshqsumner joshqsumner commented May 12, 2026

Copy link
Copy Markdown
Contributor

Describe your changes
Moving quick_color_check to qc submodule from transform

Type of update
This is a restructuring pr.

Associated issues
Part of #1910

Additional context
See #1910

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

@joshqsumner joshqsumner changed the base branch from main to v5.0 May 12, 2026 19:36
@deepsource-io

deepsource-io Bot commented May 12, 2026

Copy link
Copy Markdown

DeepSource Code Review

We reviewed changes in c17f75d...656a605 on this pull request. Below is the summary for the review, and you can see the individual issues we found as inline review comments.

See full review on DeepSource ↗

PR Report Card

Overall Grade   Security  

Reliability  

Complexity  

Hygiene  

Coverage  

Code Review Summary

Analyzer Status Updated (UTC) Details
Python Jun 15, 2026 5:28p.m. Review ↗
Code coverage Jun 15, 2026 5:28p.m. Review ↗

Code Coverage Summary

Language Line Coverage (New Code) Line Coverage (Overall)
Aggregate
100%
[✓ above threshold]
100%
Python
100%
[✓ above threshold]
100%

➟ Additional coverage metrics may have been reported. See full coverage report ↗


Important

AI Review is run only on demand for your team. We're only showing results of static analysis review right now. To trigger AI Review, comment @deepsourcebot review on this thread.

@joshqsumner joshqsumner added this to the PlantCV v5.0 milestone May 12, 2026
@joshqsumner joshqsumner added the merge in order Set of PRs that need to be merged sequentially label May 13, 2026
@nfahlgren nfahlgren merged commit 2308749 into v5.0 Jun 15, 2026
5 checks passed
@nfahlgren nfahlgren deleted the quick_color_check-to-qc branch June 15, 2026 17:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

merge in order Set of PRs that need to be merged sequentially ready to review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants