Skip to content

Conversation

@lidel
Copy link
Collaborator

@lidel lidel commented Feb 6, 2024

This PR shortens test values and adds toString column, which shows how stringified URL looks like.

This allows us to show vendors how non-HTTP URLs are mutated.

Exmample: https://github.com/SgtPooki/new-URL-protocol-hostname-path-diff/blob/ea06d9a7413f049a6c7f06b21cc5e064809ff3f7/README.md

shortens test values, limits test to long example
@lidel
Copy link
Collaborator Author

lidel commented Feb 6, 2024

@SgtPooki I was not able to run webkit tests, mind running npm run clean && npm run update-results before merging? 🙏

@lidel lidel requested a review from SgtPooki February 6, 2024 18:15
Copy link
Owner

@SgtPooki SgtPooki left a comment

Choose a reason for hiding this comment

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

lgtm. I updated the README after running npm run clean && npm run update-results as requested :)

@lidel i'm good with this if you are.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants