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

dependency: bump mime dependency to 3.0 #30966

Merged
merged 34 commits into from
Jan 30, 2025
Merged

Conversation

jennifer-shehane
Copy link
Member

@jennifer-shehane jennifer-shehane commented Jan 28, 2025

Additional details

This updates mime to 3.0. I attempted to update to 4.x and remove the types since they are included in mime now, but we need services to be an esm package completely to do that. See thread.

  • I'm not even sure how mime version 2 was working because mime notes that renaming of lookup to getType was a breaking change from v1 to v2. https://github.com/broofa/mime/tree/v2.6.0?tab=readme-ov-file#version-2-notes
  • mime in 4.x was returning text/javascript instead of application/javascript for JS files. application/javascript is obsolete according to https://www.iana.org/assignments/media-types/media-types.xhtml. But since we're staying on 3.0 - this is no longer relevant. I did however update some tests that were defining application/javascript to text/javascript - this is NOT related to the mime update at all, just figured I'd update it while I'm here, but I can pull out if needed.
  • It's a bit weird that we use both mime and mime-db, but I didn't investigate these differences at this time.

Steps to test

How has the user experience changed?

PR Tasks

Copy link

cypress bot commented Jan 28, 2025

cypress    Run #60137

Run Properties:  status check passed Passed #60137  •  git commit 1b61f92906: Merge commit 'stash' into mime-dep-update
Project cypress
Branch Review mime-dep-update
Run status status check passed Passed #60137
Run duration 17m 46s
Commit git commit 1b61f92906: Merge commit 'stash' into mime-dep-update
Committer cypress-bot[bot]
View all properties for this run ↗︎

Test results
Tests that failed  Failures 0
Tests that were flaky  Flaky 5
Tests that did not run due to a developer annotating a test with .skip  Pending 1099
Tests that did not run due to a failure in a mocha hook  Skipped 0
Tests that passed  Passing 26522
View all changes introduced in this branch ↗︎
UI Coverage  45.43%
  Untested elements 192  
  Tested elements 164  
Accessibility  92.54%
  Failed rules  3 critical   8 serious   2 moderate   2 minor
  Failed elements 888  

@jennifer-shehane jennifer-shehane self-assigned this Jan 28, 2025
@jennifer-shehane jennifer-shehane changed the title dependency: bump mime dependency dependency: bump mime dependency to 3.0 Jan 29, 2025
@jennifer-shehane jennifer-shehane requested review from ryanthemanuel and removed request for AtofStryker January 30, 2025 15:39
@jennifer-shehane jennifer-shehane merged commit 99b449d into develop Jan 30, 2025
84 of 88 checks passed
@jennifer-shehane jennifer-shehane deleted the mime-dep-update branch January 30, 2025 15:41
@cypress-bot
Copy link
Contributor

cypress-bot bot commented Feb 5, 2025

Released in 14.0.2.

This comment thread has been locked. If you are still experiencing this issue after upgrading to
Cypress v14.0.2, please open a new issue.

@cypress-bot cypress-bot bot locked as resolved and limited conversation to collaborators Feb 5, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants