Skip to content

Fix Modbus Serial - #84

Open
bobur-khay wants to merge 6 commits into
eclipse-thingweb:mainfrom
bobur-khay:fix/modbus-serial
Open

Fix Modbus Serial#84
bobur-khay wants to merge 6 commits into
eclipse-thingweb:mainfrom
bobur-khay:fix/modbus-serial

Conversation

@bobur-khay

@bobur-khay bobur-khay commented Jul 9, 2026

Copy link
Copy Markdown
Contributor

The code generator for Modbus Serial in JS had a line of code: new URL(href)

When href: "/" the new URL() was throwing an Invalid URL.

Fix:

  • when href is / it falls back to localhost now

Comment thread node/code-gen/src/generators/go.ts Fixed
@egekorkan

Copy link
Copy Markdown
Member

when href is / it falls back to localhost now

Seems to be outdated now? There is no localhost fallback I see in the code?

Also, you can bump the patch version in package.json

uses: actions/checkout@v4

- name: Install prettier
run: npm install prettier@2.3.2

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

@bobur-khay what is the reason for change? We kept having problems with prettier versioning before

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.

3 participants