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

Release #907

Merged
merged 6 commits into from
Aug 22, 2024
Merged

Release #907

merged 6 commits into from
Aug 22, 2024

Conversation

nevoodoo
Copy link
Collaborator

nevoodoo and others added 6 commits August 19, 2024 15:12
* added model changes and schema changes

* updated some logic and added first test

* fixed broken texts

* working tests, functional

* added migration script

* fixed migration script with permissions

* testing prod pipes, temp wfl change

* switch to using list operations for output files

* refactored to use prefixes when listing

* Fixed blob checks in output creation

* uncomment test gcs client

* working migration and fcf tests

* optimised client calls

* review cleanup and refactoring

* added appropriate var to mock correctly

* added monkey patch for gcs client

* removed default var

* patching all tests for gcs client

* Bump version: 7.3.3 → 7.4.0
This field is reported by the database as b'\x00' or b'\x01', both of
which bool(...) returns as True. Use parse_sql_bool() instead; and I've
verified that all other BOOLEAN or BIT fields use parse_sql_bool() too.

Add test case exercising the update_analysis() API fixed in PR #899
and checking that active is returned from the database correctly.

* Also fix typo so grepping finds this comment (trivial)
(Verified that changing this XML comment doesn't affect liquibase's
hash for this changeSet.)

* Bump version: 7.4.0 → 7.4.1
@illusional illusional merged commit 8f56bf4 into main Aug 22, 2024
6 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.

3 participants