Skip to content

Favor unverified validation; update docker compose usage - #184

Merged
ldko merged 4 commits into
masterfrom
favor-unverified-validation
Jul 28, 2025
Merged

Favor unverified validation; update docker compose usage#184
ldko merged 4 commits into
masterfrom
favor-unverified-validation

Conversation

@ldko

@ldko ldko commented Jul 7, 2025

Copy link
Copy Markdown
Member

@clarktr1 @somexpert this is ready for review. The main change of substance here is that when determining what item should be selected for validation next, bags with a last_verified_status of Unverified will be preferred over random bags that aren't prioritized and have previously been validated at some point. Also included is updating the docker compose usage to remove the outdated docker-compose command.

@ldko
ldko requested review from clarktr1 and somexpert July 7, 2025 18:03
Comment thread coda/coda_validate/tests/test_feeds.py
Comment thread coda/coda_validate/views.py

@clarktr1 clarktr1 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks okay to me!

@somexpert somexpert left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me! 👍

@ldko
ldko merged commit cdd5369 into master Jul 28, 2025
2 checks passed
@ldko
ldko deleted the favor-unverified-validation branch July 28, 2025 21:14
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