Skip to content

do-not-merge: Test PR comment - #17

Closed
joaomariolago wants to merge 1 commit into
bluerobotics:mainfrom
joaomariolago:test-pr-comment
Closed

do-not-merge: Test PR comment#17
joaomariolago wants to merge 1 commit into
bluerobotics:mainfrom
joaomariolago:test-pr-comment

Conversation

@joaomariolago

Copy link
Copy Markdown
Contributor

No description provided.

@joaomariolago
joaomariolago force-pushed the test-pr-comment branch 2 times, most recently from bfaee4b to cb6b7ff Compare June 24, 2026 00:20
Signed-off-by: Joao Mario Lago <joao.mario.lago@hotmail.com>
@github-actions

Copy link
Copy Markdown

Dictionary Validation Errors

Error: unexpected token `/*
	 A message to convey a single A-scan of sonar data
	 This doesn't contain any info about the data, like what transmit pulse was used or whatever
	 Let's assume for now that info will be transferred some other way
	 This message assumes that the  total a-scan will be sent over multiple messages
	 This message can be used as a request if captureId and firstSample index are specified and zero
	 In this case, sampleRate and data can be NAN, empty respectively, but they don't really matter
	 It is assumed that a device will not respond right away if it doesn't have data yet,
	 instead it will simply drop the message and wait for a subsequent one.
	 */`
    ╭─[./dictionary/devices/SonarMessages.idl:35:2]
    │
 35 │ ╭─▶     /*
    ┆ ┆   
 44 │ ├─▶      */
    │ │             
    │ ╰───────────── expected `;`
────╯

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.

1 participant