Skip to content

main <- dev #913

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 5 commits into from
Sep 30, 2024
Merged

main <- dev #913

merged 5 commits into from
Sep 30, 2024

Conversation

bpblanken
Copy link
Collaborator

@bpblanken bpblanken commented Sep 30, 2024

Screenshot 2024-09-30 at 4 50 50 PM

bpblanken and others added 2 commits September 30, 2024 12:34
…ion exceptions. (#899)

* Make functions error checkable

* First test passing

* whacking

* Fix tests

* testing the methods!

* another test working

* cleanup

* tweak

* Run all validations

* First pass

* ruff

* Scaffold done

* format

* ruff

* reformat

* cleanup

* cleanup

* make run id shared

* A fix

* another test

* A few more

* Few more

* Unnecessary

* ruf

* run id

* string

* unused

* missed one

* Fix it correctly

* missed one

* Last one!

* Validation refactoring

* Fix it

* case this function better

* missing comma

* change arg name

* Moving

* no yield

* Add test file

* Include VCF with multiple validation exceptions

* improve validation formatting

* Finish off test

* Remove caching block

* Fix merge

* add comments
* make updating crdqs a dependency of updating vat

* remove from init

* fix import
@bpblanken bpblanken requested a review from a team as a code owner September 30, 2024 20:51
@bpblanken bpblanken changed the title dev <- main main <- dev Sep 30, 2024
@bpblanken bpblanken merged commit 586cfed into main Sep 30, 2024
4 checks passed
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