Skip to content
This repository has been archived by the owner on Apr 19, 2024. It is now read-only.

Fix Content-Type #40

Merged
merged 1 commit into from
Feb 5, 2024
Merged

Fix Content-Type #40

merged 1 commit into from
Feb 5, 2024

Conversation

lishaduck
Copy link
Member

@lishaduck lishaduck commented Feb 5, 2024

Description

This PR makes the .pck file use the correct Content-Type.

Closes: #38


Type of Change

  • 🐛 Bug fix

Checklist

Before submitting the PR, please make sure you do the following

  • Read the Contributing Guidelines.
  • Check that there isn't already a PR that solves the problem the same way to avoid creating a duplicate.
  • Fill out this template.
  • Log your hours.
  • Check that commits follow the Angular commit convention, more or less.
  • Ideally, include relevant tests that fail without this PR but pass with it (if applicable).

Tested on

  • macOS 14

@lishaduck lishaduck added the bug Something isn't working label Feb 5, 2024
@lishaduck lishaduck self-assigned this Feb 5, 2024
@lishaduck lishaduck requested a review from a team as a code owner February 5, 2024 16:02
@lishaduck
Copy link
Member Author

I tried to add a test for compression, but that's done natively, so it can't (readily) be tested from userland. We'll just have to trust Deno with it.

@lishaduck lishaduck merged commit e77cc99 into main Feb 5, 2024
7 checks passed
@lishaduck lishaduck deleted the fix-content-type branch February 5, 2024 19:32
@lishaduck lishaduck added this to the State milestone Feb 6, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

MIME-Types and Content-Encodings are off
1 participant