Skip to content

fix: correct changelog version from 2.35.3 to 2.35.0 and add 2.35.1 entry#114

Merged
mangs merged 4 commits into
mangs:mainfrom
Jhojan28:fix/changelog-version
Jun 2, 2026
Merged

fix: correct changelog version from 2.35.3 to 2.35.0 and add 2.35.1 entry#114
mangs merged 4 commits into
mangs:mainfrom
Jhojan28:fix/changelog-version

Conversation

@Jhojan28

Copy link
Copy Markdown
Contributor

Pull Request Checklist

  • (OPTIONAL) Readme updates were made reflecting this Pull Request's changes

Changes Included

  • Correct changelog version from 2.35.3 to 2.35.0 and add 2.35.1 entry

@Jhojan28 Jhojan28 requested a review from mangs as a code owner May 20, 2026 14:25

@mangs mangs left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Looks good, otherwise needs a fix in src/filesystemUtils.mts in usingNewTemporaryFile():

line 143 needs to be prefixed with await:

await writer.write(appendContents);

If you run the linter it should flag this

Comment thread src/networkUtils.mts Outdated

interface ServerConfiguration extends Pick<ServeOptions, 'error' | 'hostname' | 'port'> {
interface ServerConfiguration
extends Partial<Pick<Serve.Options<undefined>, 'error' | 'hostname' | 'port'>> {

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Running the locally looks like this can be simplified a bit

Suggested change
extends Partial<Pick<Serve.Options<undefined>, 'error' | 'hostname' | 'port'>> {
extends Pick<Serve.Options<undefined>, 'error' | 'hostname' | 'port'> {

@mangs mangs left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Woo thanks for making this happen!

@mangs mangs merged commit d547930 into mangs:main Jun 2, 2026
1 check passed
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