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

Make it compatible with Pydantic V2 #1

Open
wants to merge 4 commits into
base: main
Choose a base branch
from
Open

Conversation

OscarMCV
Copy link

@OscarMCV OscarMCV commented Jun 6, 2024

Description

PydanticV2 has deprecated some of the decorators and methods used in the repo, generating warnings during testing.

Types of changes

Put an x in the boxes that apply.

  • Bugfix (fix): A non-breaking change which fixes an issue or bug
  • New Feature (feat): A non-breaking change which adds a new feature or new functionality
  • Tests (test): Adding missing tests or correcting existing tests
  • Refactoring (refactor): A code change that neither fixes a bug nor adds a feature, e.g Deleting unused or redundant code
  • Documentation (docs): Documentation only changes
  • Other (style, build, ci, performance): Formatting, linting or styling. Changes to the CI pipeline. Performance improvements. Changes that affect the build system.

Further Information

./usr/local/lib/python3.10/site-packages/rdbbeat/data_models.py:51
  /usr/local/lib/python3.10/site-packages/rdbbeat/data_models.py:51: PydanticDeprecatedSince20: Pydantic V1 style @validator validators are deprecated. You should migrate to Pydantic V2 style @field_validator validators, see the migration guide for more details. Deprecated in Pydantic V2.0 to be removed in V3.0. See Pydantic V2 Migration Guide at https://errors.pydantic.dev/2.6/migration/

OscarMCV and others added 4 commits June 5, 2024 20:16
The code changes in `setup.py` include adding `"SQLAlchemy-Utils"` to the `install_requires` list, indicating that the project now depends on this package.

Based on the recent repository commits, it seems that the project is actively maintained and follows a conventional commit message style.
feat: Add SQLAlchemy-Utils to project dependencies
This commit refactors the `get_crontab_schedule` function in the `controller.py` file. The function now takes a `Session` and a `Schedule` as parameters and returns a `CrontabSchedule` object. The code has been formatted for better readability.

Based on the recent repository commits, it seems that the project follows a conventional commit message style.
feat: Refactor get_crontab_schedule function
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.

1 participant