I create software, rock operas, and pizzas. I am a member of the Node.js Technical Steering Committee and work for the University of California. he/him
- San Francisco, CA
- @Trott
Highlights
- 4 discussions answered
Block or Report
Block or report Trott
Report abuse
Contact GitHub support about this user’s behavior. Learn more about reporting abuse.
Report abuse4,012 contributions in the last year
Less
More
Activity overview
Contribution activity
March 2022
Created 41 commits in 14 repositories
Created a pull request in nodejs/node that received 23 comments
url: trim leading and trailing C0 control chars
Emulate the WHATWHG URL parse behavior of trimming leading and trailing C0 control characters. This moves url.parse() slightly closer to WHATWHG UR…
+16
−6
•
23
comments
Opened 30 other pull requests in 13 repositories
nodejs/node
2
open
6
merged
1
closed
- doc: improve README.md usability
- doc: remove unneeded lint disable comment
- worker: do not send message if port is closing
- doc: revise async_hooks docs
- url: throw on NULL in IPv6 hostname
- build: skip ASan if only tests are modified
- test: add test case for reverted 17.7 regression
- url: preserve null char in WHATWG URL errors
- doc: remove "considered" for clarity
nodejs/nodejs.org
1
open
1
closed
4
merged
nodejs/admin
2
merged
Trott/slug
2
merged
ucsf-ckm/som-qualtrics-survey-responses
2
merged
Trott/music-routing
2
merged
nodejs/node-core-utils
1
merged
Trott/dotfiles
1
merged
nodejs/TSC
1
merged
nodejs/email
1
merged
nodejs/security-wg
1
merged
Trott/remark-lint-prohibited-strings
1
merged
Trott/solr-proxy
1
merged
Reviewed 66 pull requests in 8 repositories
nodejs/node
44 pull requests
- repl,worker: fix crash when SharedArrayBuffer disabled
- [v14.x] update to OpenSSL 1.1.1n
- doc: add @ShogunPanda to collaborators
- doc: update base branch name for nodejs org
- src,doc: Experimental support for SEA
- doc: update security release onboarding
- doc: document goal to have examples
- test: add test for exception handlings in debugger
- doc: clarify the meaning of legacy status
- meta: update AUTHORS
- tools: update eslint to 8.11.0
- tools: update lint-md-dependencies to rollup@2.70.0 unified@10.1.2
- tools: update doc to highlight.js@11.5.0 unified@10.1.2
- src: check EC_POINT_get_affine_coordinates result
- src: check return value of HMAC_Final
- [16.x] build: remove windows-2022 from v16.x actions
- doc: improve pipe description
- build: bump macOS deployment target to 10.15
- deps: upgrade npm to 8.5.4
- v17.7.1 release proposal
-
url: revert fix url.parse() for
@hostname - src: fix coverity warnings in node_file.cc
- url: preserve null char in WHATWG URL errors
- debugger: correct typo in inspect_repl.js
- esm, doc: fix and improve specifier resolution warning
- Some pull request reviews not shown.
nodejs/nodejs.org
12 pull requests
- Blog: v16.14.1 release post
- fix: blog banner link
- doc: add security release to banner
- doc: minor update on the timing of sec release this week
- doc: mar-2022 OpenSSL security pre-release announcement
- build(deps-dev): bump lockfile-lint from 4.6.2 to 4.7.4
- chore: update eslint@7.32.0 to 8.10.0
- Added theme changing transition.
- build(deps): bump postcss from 8.4.7 to 8.4.8
- Fixed typo (bug in code example)
- Modified StandWithUkraine PR
-
🇺🇦 Support Ukraine
nodejs/TSC
3 pull requests
nodejs/node-core-utils
2 pull requests
nodejs/admin
2 pull requests
openjs-foundation/cross-project-council
1 pull request
nodejs/core-validate-commit
1 pull request
ilios/iliosproject.org
1 pull request
Created an issue in nodejs/build that received 3 comments
Access to arm32 build machines for @ShogunPanda
From @ShogunPanda: @nodejs/build Some tests is failing on ARM but not on my machine. May I get access to the CI machine? Originally posted by @Sh…
3
comments