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

url: revokeObjectURL should throw an error when no parameter is provided #50925

Closed
wants to merge 3 commits into from

Conversation

strawberrywz
Copy link

I failed to follow the guidelines for my first commit. I made a typo error so I can reproduce the code and submit it with the correct guidelines. Thank you !

@nodejs-github-bot
Copy link
Collaborator

Review requested:

  • @nodejs/url

@nodejs-github-bot nodejs-github-bot added needs-ci PRs that need a full CI run. whatwg-url Issues and PRs related to the WHATWG URL implementation. labels Nov 26, 2023
Copy link
Member

@himself65 himself65 left a comment

Choose a reason for hiding this comment

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

This looks good since it fits with the web API. But You might need to add a test case

image

Copy link
Member

@himself65 himself65 Nov 26, 2023

Choose a reason for hiding this comment

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

This file is not supposed to have. Please remove it

Copy link
Author

Choose a reason for hiding this comment

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

How do I add a test case? I also see an error which says the following:
Check failure on line 1107 in lib/internal/url.js
GitHub Actions
/ lint-js-and-md
Expected '===' and instead saw '=='

am I to fix this as well or?

@himself65 himself65 changed the title First Time Contributor url: revokeObjectURL should throw an error when no parameter is provided Nov 26, 2023
@meyfa
Copy link
Contributor

meyfa commented Nov 26, 2023

This fixes issue #50432, which could be added to the PR description.

@deokjinkim
Copy link
Contributor

Duplicate of #50433

@deokjinkim deokjinkim marked this as a duplicate of #50433 Nov 28, 2023
@marco-ippolito marco-ippolito added the duplicate Issues and PRs that are duplicates of other issues or PRs. label Dec 21, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
duplicate Issues and PRs that are duplicates of other issues or PRs. needs-ci PRs that need a full CI run. whatwg-url Issues and PRs related to the WHATWG URL implementation.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants