Skip to content

oelint: drop the space at the first line in a multiline assignment - #317

Merged
robwoolley merged 1 commit into
ros-infrastructure:masterfrom
vermaete:vermaete/oelint
Aug 20, 2026
Merged

oelint: drop the space at the first line in a multiline assignment#317
robwoolley merged 1 commit into
ros-infrastructure:masterfrom
vermaete:vermaete/oelint

Conversation

@vermaete

@vermaete vermaete commented Jul 6, 2025

Copy link
Copy Markdown
Contributor

This fixes this issue with the oelint-adv tool:
info:oelint.vars.notneededspace:Space at the beginning of the var is not needed [branch:true]

@vermaete

Copy link
Copy Markdown
Contributor Author

@robwoolley Do you need something to be done by me before merging?

This fixes this issue with the oelint-adv tool:
  info:oelint.vars.notneededspace:Space at the beginning of the var is not needed [branch:true]

Signed-off-by: Jan Vermaete <jan.vermaete@gmail.com>
@robwoolley
robwoolley self-requested a review August 20, 2026 01:27

@robwoolley robwoolley left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

This change looks good.

@robwoolley
robwoolley merged commit b796a36 into ros-infrastructure:master Aug 20, 2026
10 checks passed
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.

2 participants