Skip to content

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

Open
Jhojan28 wants to merge 3 commits into
mangs:mainfrom
Jhojan28:fix/changelog-version
Open

fix: correct changelog version from 2.35.3 to 2.35.0 and add 2.35.1 entry#114
Jhojan28 wants to merge 3 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
Copy link
Copy Markdown
Owner

@mangs mangs left a comment

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

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'> {

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