Skip to content

version 11 release #542

Merged
joshdimanteto merged 133 commits into
mainfrom
develop
Apr 30, 2026
Merged

version 11 release #542
joshdimanteto merged 133 commits into
mainfrom
develop

Conversation

@joshdimanteto

Copy link
Copy Markdown
Contributor

Description

v11 release

Testing Instructions

Add a set up instructions describing how the reviewer should test the code

  • Review code
  • Check GitHub Actions build
  • If icatdb Generator Script Consistency Test CI job fails, is this because of a deliberate change made to the script to change generated data (which isn't actually a problem) or is here an underlying issue with the changes made?
  • Review changes to test coverage
  • Does this change mean a new patch, minor or major version should be made? If so, does one of the commit messages feature fix:, feat: or BREAKING CHANGE: so a release is automatically made via GitHub Actions upon merge?
  • {more steps here}

- remove downgrade of setuptools
Added `# noqa: B907` in several places where manual quoting is required
for valid SQL query syntax. Using `!r` would change the format and break
query parsing, so explicit quotes are necessary. This ensures tests and
API calls remain correct while satisfying linting requirements.
- In the test_query_filter_factory the test don't work on the CI as it
  is not setting up the db backend.
joshdimanteto and others added 26 commits March 12, 2026 12:52
Merge main into develop 10.0.0
…pydantic-522

Refactor dg api model to pydantic #522
- only changes in src directory
- improve formatting in test files
- Rename debug_mode to reload
- Change the type of port from string to integer
BREAKING CHANGE: Flask-RESTful has been removed and replaced with FastAPI.
@joshdimanteto joshdimanteto marked this pull request as ready for review April 30, 2026 09:27

@patrick-austin patrick-austin left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

I've reviewed the commit history and am satisfied that all these changes come from commits that were reviewed in previously approved PRs (#507 #521 #523 #524 #537), so on that basis happy to approve this.

@joshdimanteto joshdimanteto merged commit 3989a65 into main Apr 30, 2026
10 of 11 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