Skip to content

Fix docs about the A param of KSN config #1803

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
Open
wants to merge 1 commit into
base: feat-ros-8.2
Choose a base branch
from
Open

Fix docs about the A param of KSN config #1803

wants to merge 1 commit into from
+1 −1

Conversation

minchopaskal
Copy link
Contributor

In recent PR about adding docs for new KSN types I forgot to add some docs, so this PR fixes that

@minchopaskal
Copy link
Contributor Author

@dwdougherty sorry for the mistake! Pls review

@minchopaskal
Copy link
Contributor Author

@dwdougherty actually, this became a point of dialogue if these new KSN even types should be enabled by default for the A param. I'll let u know if it's ready to be merged or I'll close it if not.

@dwdougherty dwdougherty self-assigned this Jul 3, 2025
dwdougherty
dwdougherty approved these changes Jul 3, 2025
View reviewed changes
Copy link
Collaborator

@dwdougherty dwdougherty left a comment

Choose a reason for hiding this comment

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

LGTM. I'll hold until you tell me to let it fly.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@dwdougherty dwdougherty dwdougherty approved these changes

Assignees

@dwdougherty dwdougherty

Labels
dev do not merge yet ros 8.2
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@minchopaskal @dwdougherty