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.

Sign up for GitHub

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

Jump to bottom

DM-47811: Fix FAST001 and FAST002 ruff rules and update-deps #19

Merged

stvoutsin merged 1 commit into main from tickets/DM-47811

Nov 27, 2024

Member

stvoutsin commented Nov 26, 2024 •

edited

Loading

Summary

Fix ruff rules FAST001 & FAST002 introduced in ruff version 0.8.0

This PR addresses this in the following way:

Updates the dependencies including ruff
Removes the response_model param which is no longer needed
Remove pydantic_xml.mypy from pyproject ruff config (otherwise mypy typing check fails)
Use lsst-dax-obscore pip package instead of github repo

Tested

tox
end-end test on data-dev with locally built Docker image

stvoutsin marked this pull request as ready for review

November 26, 2024 22:22


          Remove response_model for ruff 0.8.0 and update-deps

e0a466e

stvoutsin force-pushed the tickets/DM-47811 branch from 57c77d5 to e0a466e Compare

November 27, 2024 17:58

stvoutsin merged commit c4e61fa into main

5 checks passed

stvoutsin mentioned this pull request

DM-48121: Upgrade SIA application to v0.1.4 lsst-sqre/phalanx#3981

Open

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet