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

feat: add file type validation and mime type extraction #1893

Merged

eapolinario merged 50 commits into flyteorg:master from jasonlai1218:add-validation-flyteformat-type

Nov 16, 2023

Merged

feat: add file type validation and mime type extraction #1893

test: add test for invalid file type validation

Codecov / codecov/patch failed Oct 19, 2023 in 0s

25.00% of diff hit (target 55.23%)

View this Pull Request on Codecov

25.00% of diff hit (target 55.23%)

Annotations

Check warning on line 325 in flytekit/types/file/file.py

codecov / codecov/patch

flytekit/types/file/file.py#L325

Added line #L325 was not covered by tests

Check warning on line 340 in flytekit/types/file/file.py

codecov / codecov/patch

flytekit/types/file/file.py#L340

Added line #L340 was not covered by tests

Check warning on line 345 in flytekit/types/file/file.py

codecov / codecov/patch

flytekit/types/file/file.py#L344-L345

Added lines #L344 - L345 were not covered by tests

Check warning on line 347 in flytekit/types/file/file.py

codecov / codecov/patch

flytekit/types/file/file.py#L347

Added line #L347 was not covered by tests

Check warning on line 373 in flytekit/types/file/file.py

codecov / codecov/patch

flytekit/types/file/file.py#L373

Added line #L373 was not covered by tests

Check warning on line 399 in flytekit/types/file/file.py

codecov / codecov/patch

flytekit/types/file/file.py#L399

Added line #L399 was not covered by tests

View more details on Codecov