Skip to content

[chore] number of minor updates/modernizations to jsg code#5683

Merged
jasnell merged 11 commits intomainfrom
jasnell/modernize-jsg-webidl
Dec 12, 2025
Merged

[chore] number of minor updates/modernizations to jsg code#5683
jasnell merged 11 commits intomainfrom
jasnell/modernize-jsg-webidl

Conversation

@jasnell
Copy link
Copy Markdown
Collaborator

@jasnell jasnell commented Dec 12, 2025

This is a hodgepodge of minor updates to various parts of the jsg code to modernize a bit where appropriate, fixup minor issues (like typos, duplications, dead code, etc), de-deduplicate a bit, etc. I'd been working through a few of these manually but decided to see if claude could handle it and it actually did a pretty good job. About half of the suggested improvements it came up with were fairly silly tho so I ended up ignoring about half of what it came up with. Still, a good exercise. The vast majority of these will not be functional changes and those that are should only impact compile time.

@jasnell jasnell requested review from a team as code owners December 12, 2025 02:13
@jasnell jasnell added the technical debt Label used for changes that address technical debt label Dec 12, 2025
@jasnell jasnell requested review from fhanau and npaun December 12, 2025 02:13
@jasnell jasnell merged commit e872176 into main Dec 12, 2025
20 of 21 checks passed
@jasnell jasnell deleted the jasnell/modernize-jsg-webidl branch December 12, 2025 17:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

technical debt Label used for changes that address technical debt

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants