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.

Sign up for GitHub

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

Jump to bottom

fix: canboat analyzerOutEvent event name & minor fixes #1846

Merged
merged 5 commits into from
Dec 15, 2024
Merged

fix: canboat analyzerOutEvent event name & minor fixes #1846

merged 5 commits into from
Dec 15, 2024

Conversation

gabor-toth
Copy link
Contributor

@gabor-toth gabor-toth commented Dec 11, 2024 •
edited by tkurki
Loading

  • fix analyzerOutEvent event name so that raw analyzer output is emitted correctly when using canboat
  • output canboat version in server log on startup
  • ignore IDEA project file
  • fix accidental global variable

@tkurki tkurki added the fix label Dec 14, 2024
@tkurki tkurki changed the title fix emitter name in n2kAnalyzer fix: canboat analyzerOutEvent event name & minor fixes Dec 14, 2024
@tkurki
Copy link
Member

tkurki commented Dec 14, 2024

I rewrote the title and description. Could you please fix the formatting issues? Run npm run format in the streams directory.

@gabor-toth
Copy link
Contributor Author

code formatting applied

@tkurki tkurki merged commit 8102ec5 into SignalK:master Dec 15, 2024
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
fix
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@gabor-toth @tkurki