Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: resource-provider server-api dependency #1824

Merged
merged 1 commit into from
Oct 30, 2024

Conversation

panaaj
Copy link
Member

@panaaj panaaj commented Oct 29, 2024

Add @signalk/server-api as a dependency rather than a dev-dependency.

This fix is included as part of v1.2.1 release.

rather than a dev-dependency
@panaaj panaaj added the fix label Oct 29, 2024
@tkurki
Copy link
Member

tkurki commented Oct 30, 2024

I thougth server api was just typescript and not used in runtime at all. Why is it needed?

@panaaj
Copy link
Member Author

panaaj commented Oct 30, 2024

I thougth server api was just typescript and not used in runtime at all. Why is it needed?

v1.2.0 released to npmjs (dev-dependency), the plugin failed to load when the server starts citing unable to find sever-api references.
v1.2.1 (with with server-api as dependency) fixed the issue.

The Freeboard-SK plugin also needs it as a dependency also, there clearly a runtime requirement created by tsc.

@tkurki
Copy link
Member

tkurki commented Oct 30, 2024

I see some consts and a class now, not just types and interfaces.

@tkurki tkurki added chore and removed fix labels Oct 30, 2024
@tkurki tkurki merged commit db5c3a0 into master Oct 30, 2024
5 checks passed
@tkurki tkurki deleted the resources-plugin-package-fix branch October 30, 2024 06:22
@tkurki
Copy link
Member

tkurki commented Oct 30, 2024

Changed the label to chore so that this does not get grouped under fixes in release notes.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants