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

Schematic v24.10.1 #1514

Merged
merged 274 commits into from
Oct 4, 2024
Merged

Schematic v24.10.1 #1514

merged 274 commits into from
Oct 4, 2024

Conversation

lakikowolfe
Copy link
Contributor

@lakikowolfe lakikowolfe commented Oct 3, 2024

Schematic pypi release

FDS-2297

Release Scope

linglp and others added 21 commits September 18, 2024 10:29
fix: call get_node_validation_rules with appropriate parameters when storing annotations
* test single version

* remove python version from name

* remove feature branch pdoc run

* remove quotes
* splits up test_filename_manifest

* adds unit tests

* updates test_validation

* removes commented code

* fix test order

* revert test_validation changes

* revert id

* updates filename_validation and testing

* removes extra comment

* updates tests to txt6

* test python version issue

* undo last

* redo last

* fix pipe

* run pytest in line

* try poetry shell

* removes caching

* updates unit tests for dataset_scope

* try python version setting

* split lines

* revert ci test

* adds +2 comment

* accounts for entityId empty strings

* use existing DMGE fixture

* corrects val_rule

* dummy commit

* dummy commit

* test without caching

* remvoe caching

* string version

* remove caching

* remove test caching

* try caching path

* remove caching key version

* try jenny fix for venv corruption
* update how annotations are accessed

* update submission integration test

* Remove venv caching

---------

Co-authored-by: Jenny Medina <medina.stsci@gmail.com>
Co-authored-by: Jenny V Medina <jenny.medina@sagebase.org>
Fix: remove caching poetry dependencies in CI workfows
Copy link

sonarqubecloud bot commented Oct 4, 2024

@thomasyu888
Copy link
Member

I would ignore the SonarCloud Code analysis, but awesome to see the integration tests passed!

@lakikowolfe lakikowolfe merged commit d4dfaf5 into main Oct 4, 2024
12 of 13 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.

8 participants