anonrig
Follow
Pinned
3,320 contributions in the last year
Activity overview
Contribution activity
March 2023
Created 36 commits in 6 repositories
Created a pull request in nodejs/node that received 15 comments
url: fix array overrun in node:url::SetArgs()
I am following up on @tniessen's pull request and @bnoordhuis's comments. @TimothyGu, thanks for the mention & reminder Reference: #46541 (comment)
+17
−41
•
15
comments
Opened 39 other pull requests in 5 repositories
ada-url/ada
21
merged
1
closed
1
open
- fix: handle base url authority setter properly
- fix: improve url_components consistency check
- fix: improve indexing errors
- fix: try to fix password setters
- feat: add search setters for aggregator
- Update web platform tests
- feat: add authority state parsing
- feat: add hostname setters to aggregator
- feat: add protocol setters
- test: add tests for url_aggregator
- test: add tests that cover url::aggregator
- test: improve tests by adding templates
- fix: remove unused encoding parameter
- feat: add templates for aggregator and url
- Update web platform tests
- feat: use pure virtual functions
- feat: add more scheme related functions to url_base
- fix: add more internal getters
- feat: add several setters
- feat: propose url_aggregator
- perf: improve url components getter
- perf: improve href getter performance
- feat: add url_components for start/end
nodejs/node
5
open
5
merged
1
closed
- Improve webstreams brand validations
- url: conform structuredClone url serialization
-
url: add pending-deprecation to
url.parse() - lib: runtime deprecate punycode
- util: reduce javascript call for ToUSVString
- url: improve URLSearchParams creation performance
- test: update URL web-platform-tests
-
src: remove unused
v8::Uint32Arrayfrom encoding - test: remove unnecessary urlToHttpOptions test
- benchmark: add real-world examples for url parsing
- url: use private properties for brand check
nodejs/nodejs.org
1
open
1
merged
1
closed
anonrig/fast-querystring
1
merged
web-infra-dev/rspack
1
open
Reviewed 107 pull requests in 10 repositories
nodejs/node
25 pull requests
- util: reduce javascript call for ToUSVString
- test,crypto: update WebCryptoAPI WPT
- test: remove useless WPT init scripts
- bootstrap: store internal loaders in C++ via a binding
-
url: add pending-deprecation to
url.parse() - lib: runtime deprecate punycode
- tools: automate brotli update
- url: implement URL.canParse
- buffer: graduate File from experimental and expose as global
- lib: define Event.isTrusted in the prototype
- tools: notify on Slack when invalid commit lands
- test: unused callback variables removed
- deps: update corepack to 0.17.1
-
util: fix inspecting error with a throwing getter for
cause - buffer: use private properties for brand checks in File
- src: stop copying code cache
- url: allow extension of user provided URL objects
- stream: don't wait for next item in take when finished
- test: mark test-http-max-sockets as flaky on win32
- doc: rename the startup performance initiative to startup snapshot
- POC: use JSON configuration and blob content for SEA
-
doc: clarify that
fs.create{Read,Write}StreamsupportAbortSignal - deps: upgrade npm to 8.19.4
- deps: upgrade npm to 9.6.2
-
doc: indicate that
nameis no longer an optional argument - Some pull request reviews not shown.
ada-url/ada
25 pull requests
- empty host slash dot
- various simplifications to dev branch
- Various minor fixes to ada::url_aggregator
- This adds some optimization to ada::url_aggregator
- Fixes clear path and adds many validate
- Clang format
- adding ada dev checks
- optimization: reserving memory for url_allocator
- Testing empty and dash URLs (base-less).
- optimization: This PR moves the insertion of the hash last
- feat: add authority state parsing
- Various comments on feat-add-protocol-setters
- test: add tests for url_aggregator
- refactor: undoing inheritance
- fix: remove unused encoding parameter
- Fix: compilation of proposal-aggregator
- feat: use pure virtual functions
- fix the underlying type of scheme::type
- Alternative design
- fix: standardize the benchmarks
- Add vs code dev container support
- feat: add new data benchmark (100k URLs)
- avoid needless to_string(x).size()
- Some proposed changes to feat: add url_components for start/end
- Additional tests related to pathToFileURL
nodejs/node-core-utils
6 pull requests
-
feat(git-node): add support for the
--gpg-signgit flag - feat(git-node): allow empty commits with fixupAll
- fix(pr_checker): do not count non-approving reviews
-
fix: do not run
git cherry-pick --aborton failure - fix(wpt): remove stale fixtures before pulling fresh ones
-
fix: treat
fast-trackwith not enough approvals as non-fatal
nodejs/nodejs.org
5 pull requests
ada-url/idna
2 pull requests
openjs-foundation/cross-project-council
2 pull requests
nodejs/undici
2 pull requests
calcom/cal.com
1 pull request
nodejs/node-gyp
1 pull request
ada-url/url-dataset
1 pull request
Created an issue in corrosion-rs/corrosion that received 8 comments
Package does not play well with ccache and ninja
Here's the full error message
[2/58] cd /Users/yagiz/Developer/url-parser/benchmarks/competitors/servo-url &...parser/build/./cargo/build --profile…
8
comments
Opened 3 other issues in 3 repositories
nodejs/build
1
open
nodejs/node
1
closed
nodejs/node-core-utils
1
open
Started 1 discussion in 1 repository
nodejs/node
nodejs/node
-
[Research] Communication steps between JS and C++ in core
This contribution was made on Mar 16
4
contributions
in private repositories
Mar 3 – Mar 23