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

chore: replace deprecated typeByExtension() in content_type.ts #3727

Merged
merged 2 commits into from
Oct 28, 2023

Conversation

0scvr
Copy link
Contributor

@0scvr 0scvr commented Oct 26, 2023

Replaced call to deprecated typeByExtension in content_type.ts in favour of private typeByExtension function.

See #3718

Not sure if this is what you were looking for. Seems odd to deprecate typeByExtension when it is needed by content type and would need to be reimplemented for content type when removed.

@iuioiua
Copy link
Contributor

iuioiua commented Oct 26, 2023

Ah, these were the types of issues I was hoping to catch when I first created the issue. I’m not certain what to do here.

Either way, can you please fix the failing tests?

@0scvr
Copy link
Contributor Author

0scvr commented Oct 26, 2023

Ah, these were the types of issues I was hoping to catch when I first created the issue. I’m not certain what to do here.

Either way, can you please fix the failing tests?

Yes, my mistake. I forgot to import types.

@iuioiua iuioiua changed the title chore: Replace deprecated typeByExtension in content_type.ts chore: replace deprecated typeByExtension() in content_type.ts Oct 28, 2023
Copy link
Contributor

@iuioiua iuioiua left a comment

Choose a reason for hiding this comment

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

On second thought, LGTM! Thank you, Oscar.

@iuioiua iuioiua merged commit 22cc48d into denoland:main Oct 28, 2023
12 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