Skip to content

[pull] main from nodejs:main#2946

Merged
pull[bot] merged 3 commits intoSimenB:mainfrom
nodejs:main
Feb 3, 2025
Merged

[pull] main from nodejs:main#2946
pull[bot] merged 3 commits intoSimenB:mainfrom
nodejs:main

Conversation

@pull
Copy link
Copy Markdown

@pull pull bot commented Feb 3, 2025

See Commits and Changes for more details.


Created by pull[bot] (v2.0.0-alpha.1)

Can you help keep this open source service alive? 💖 Please sponsor : )

Signed-off-by: James M Snell <jasnell@gmail.com>
PR-URL: #56792
Reviewed-By: Yagiz Nizipli <yagiz@nizipli.com>
PR-URL: #56870
Reviewed-By: Juan José Arboleda <soyjuanarbol@gmail.com>
Reviewed-By: Rafael Gonzaga <rafael.nunu@hotmail.com>
Reviewed-By: Jordan Harband <ljharb@gmail.com>
Reviewed-By: Matteo Collina <matteo.collina@gmail.com>
* remove USE from node_url_pattern to handle error correctly
* simplify if block in node_url_pattern
* improve error handling in node_url_pattern
* fix error propagation on URLPattern init
* use ToV8Value where more convenient in node_url_pattern
* simplify URLPatternInit::ToJsObject by reducing boilerplate

PR-URL: #56871
Reviewed-By: Yagiz Nizipli <yagiz@nizipli.com>
Reviewed-By: Franziska Hinkelmann <franziska.hinkelmann@gmail.com>
Reviewed-By: Chengzhong Wu <legendecas@gmail.com>
@pull pull bot added the ⤵️ pull label Feb 3, 2025
@pull pull bot merged commit 35d199f into SimenB:main Feb 3, 2025
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.

1 participant