Skip to content

Conversation

@franciscojavierarceo
Copy link
Member

What this PR does / why we need it:

Skip tests on doc updates

Which issue(s) this PR fixes:

#4568

Misc

devin-ai-integration bot and others added 4 commits December 11, 2024 03:47
- Add deprecation warning when value_type is not specified for an entity
- Add test cases to verify deprecation warning behavior
- Prepare for making value_type mandatory in next release

Issue: feast-dev#4670

Co-Authored-By: Francisco Javier Arceo <arceofrancisco@gmail.com>
Signed-off-by: Devin AI <158243242+devin-ai-integration[bot]@users.noreply.github.com>
- Reorder imports according to PEP8
- Group standard library imports together
- Fix ruff linting issues

Co-Authored-By: Francisco Javier Arceo <arceofrancisco@gmail.com>
Signed-off-by: Devin AI <158243242+devin-ai-integration[bot]@users.noreply.github.com>
…atory-entity-value-type

feat: Make entity value_type mandatory with deprecation warning
Co-Authored-By: Francisco Javier Arceo <arceofrancisco@gmail.com>
@dmartinol
Copy link
Contributor

the fix also includes commits that are part of other PRs. is there a way to configure devin for using different branches and separate the PRs?

@franciscojavierarceo
Copy link
Member Author

the fix also includes commits that are part of other PRs. is there a way to configure devin for using different branches and separate the PRs?

I asked it to make this PR after it did this one: #4833

So I guess it didn't think to make a new branch. I'll merge this after I merge that PR.

@dmartinol dmartinol merged commit d720cdf into feast-dev:master Dec 11, 2024
27 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants