Skip to content

has whitespace error#32

Closed
cottsay wants to merge 1 commit intocottsay/per-commit-reviewsfrom
cottsay/rapid-iteration
Closed

has whitespace error#32
cottsay wants to merge 1 commit intocottsay/per-commit-reviewsfrom
cottsay/rapid-iteration

Conversation

@cottsay
Copy link
Owner

@cottsay cottsay commented Feb 13, 2026

No description provided.

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

Thanks for sending a pull request to ROS distro!

This is an automated tool that helps check your pull request for correctness.
This tool checks a number of attributes associated with your ROS package and generates a report that helps our reviewers merge your pull request in a timely fashion. Here are a few things to consider when sending adding or updating a package to ROS Distro.
ROS Distro includes a very helpful CONTRIBUTING.md file that we recommend reading if it is your first time submitting a package.
Please also read the ROS Distro review guidelines which summarizes this release process.

ROS Distro Considerations

Package Considerations

Having your package included in a ROS Distro is a badge of quality, and we recommend that package developers strive to create packages of the highest quality. We recommend package developers review the following resources before submitting their package.

Need Help?

Please post your questions to Robotics Stack Exchange or refer to the Infrastructure General channel on our Zulip server.


For changes related to rosdep:

  • ✅ New rosdep keys are named appropriately
  • ✅ Platforms for new rosdep rules are valid
  • ✅ New keys appear suitable for rosdep

For changes related to yamllint:

  • ❌ One or more linter violations were added to YAML files

rosdep/base.yaml Outdated
@@ -1,3 +1,6 @@
0ad:
fedora: [0ad]

Choose a reason for hiding this comment

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

This line does not pass YAML linter checks: trailing spaces

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

For changes related to rosdep:

  • ✅ New rosdep keys are named appropriately
  • ✅ Platforms for new rosdep rules are valid
  • ✅ New keys appear suitable for rosdep

For changes related to yamllint:

  • ❌ One or more linter violations were added to YAML files

rosdep/base.yaml Outdated
@@ -1,3 +1,6 @@
0ad:
fedora: [0ad]

Choose a reason for hiding this comment

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

This line does not pass YAML linter checks: trailing spaces

Copy link
Collaborator

@cottsay-alt cottsay-alt left a comment

Choose a reason for hiding this comment

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

For changes related to rosdep:

  • ✅ New rosdep keys are named appropriately
  • ✅ Platforms for new rosdep rules are valid
  • ✅ New keys appear suitable for rosdep

For changes related to yamllint:

  • ❌ One or more linter violations were added to YAML files

rosdep/base.yaml Outdated
@@ -1,3 +1,6 @@
0ad:
fedora: [0ad]
Copy link
Collaborator

Choose a reason for hiding this comment

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

This line does not pass YAML linter checks: trailing spaces

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

For changes related to rosdep:

  • ✅ New rosdep keys are named appropriately
  • ✅ Platforms for new rosdep rules are valid
  • ✅ New keys appear suitable for rosdep

For changes related to yamllint:

  • ✅ All new lines of YAML pass linter checks

Copy link
Collaborator

@cottsay-alt cottsay-alt left a comment

Choose a reason for hiding this comment

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

For changes related to rosdep:

  • ✅ New rosdep keys are named appropriately
  • ✅ Platforms for new rosdep rules are valid
  • ✅ New keys appear suitable for rosdep

For changes related to yamllint:

  • ✅ All new lines of YAML pass linter checks

Copy link
Collaborator

@cottsay-alt cottsay-alt left a comment

Choose a reason for hiding this comment

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

For changes related to rosdep:

  • ✅ New rosdep keys are named appropriately
  • ✅ Platforms for new rosdep rules are valid
  • ✅ New keys appear suitable for rosdep

For changes related to yamllint:

  • ✅ All new lines of YAML pass linter checks

Copy link
Collaborator

@cottsay-alt cottsay-alt left a comment

Choose a reason for hiding this comment

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

For changes related to rosdep:

  • ✅ New rosdep keys are named appropriately
  • ✅ Platforms for new rosdep rules are valid
  • ✅ New keys appear suitable for rosdep

For changes related to yamllint:

  • ✅ All new lines of YAML pass linter checks

Copy link
Collaborator

@cottsay-alt cottsay-alt left a comment

Choose a reason for hiding this comment

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

For changes related to rosdep:

  • ✅ New rosdep keys are named appropriately
  • ✅ Platforms for new rosdep rules are valid
  • ✅ New keys appear suitable for rosdep

For changes related to yamllint:

  • ❌ One or more linter violations were added to YAML files

rosdep/base.yaml Outdated
@@ -1,3 +1,6 @@
0ad:
fedora: [0ad]
Copy link
Collaborator

Choose a reason for hiding this comment

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

This line does not pass YAML linter checks: trailing spaces

@cottsay cottsay force-pushed the cottsay/rapid-iteration branch from ede1900 to a2ad82e Compare February 14, 2026 00:20
Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

For changes related to rosdep:

  • ✅ New rosdep keys are named appropriately
  • ✅ Platforms for new rosdep rules are valid
  • ✅ New keys appear suitable for rosdep

For changes related to yamllint:

  • ❌ One or more linter violations were added to YAML files

rosdep/base.yaml Outdated
@@ -1,3 +1,6 @@
0ad:
fedora: [0ad]

Choose a reason for hiding this comment

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

This line does not pass YAML linter checks: trailing spaces

Copy link
Collaborator

@cottsay-alt cottsay-alt left a comment

Choose a reason for hiding this comment

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

For changes related to rosdep:

  • ✅ New rosdep keys are named appropriately
  • ✅ Platforms for new rosdep rules are valid
  • ✅ New keys appear suitable for rosdep

For changes related to yamllint:

  • ❌ One or more linter violations were added to YAML files

rosdep/base.yaml Outdated
@@ -1,3 +1,6 @@
0ad:
fedora: [0ad]
Copy link
Collaborator

Choose a reason for hiding this comment

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

This line does not pass YAML linter checks: trailing spaces

Copy link
Collaborator

@cottsay-alt cottsay-alt left a comment

Choose a reason for hiding this comment

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

For changes related to rosdep:

  • ✅ New rosdep keys are named appropriately
  • ✅ Platforms for new rosdep rules are valid
  • ✅ New keys appear suitable for rosdep

For changes related to yamllint:

  • ❌ One or more linter violations were added to YAML files

rosdep/base.yaml Outdated
@@ -1,3 +1,6 @@
0ad:
fedora: [0ad]
Copy link
Collaborator

Choose a reason for hiding this comment

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

This line does not pass YAML linter checks: trailing spaces

Copy link
Collaborator

@cottsay-alt cottsay-alt left a comment

Choose a reason for hiding this comment

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

For changes related to rosdep:

  • ✅ New rosdep keys are named appropriately
  • ✅ Platforms for new rosdep rules are valid
  • ✅ New keys appear suitable for rosdep

For changes related to yamllint:

  • ❌ One or more linter violations were added to YAML files

fedora: [0ad]
ubuntu: [0ad]
ace:
debian: [libace-dev]
Copy link
Collaborator

Choose a reason for hiding this comment

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

This line does not pass YAML linter checks: trailing spaces

@cottsay cottsay force-pushed the cottsay/rapid-iteration branch from a2ad82e to 72a0115 Compare February 14, 2026 02:35
Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

For changes related to rosdep:

  • ✅ New rosdep keys are named appropriately
  • ✅ Platforms for new rosdep rules are valid
  • ✅ New keys appear suitable for rosdep

For changes related to yamllint:

  • ✅ All new lines of YAML pass linter checks

Copy link
Collaborator

@cottsay-alt cottsay-alt left a comment

Choose a reason for hiding this comment

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

For changes related to rosdep:

  • ✅ New rosdep keys are named appropriately
  • ✅ Platforms for new rosdep rules are valid
  • ✅ New keys appear suitable for rosdep

For changes related to yamllint:

  • ❌ One or more linter violations were added to YAML files

rosdep/base.yaml Outdated
@@ -1,3 +1,6 @@
0ad:
fedora: [0ad]
Copy link
Collaborator

Choose a reason for hiding this comment

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

This line does not pass YAML linter checks: trailing spaces

@cottsay cottsay force-pushed the cottsay/per-commit-reviews branch from 701ede8 to 56e3022 Compare February 14, 2026 02:48
@cottsay cottsay force-pushed the cottsay/rapid-iteration branch from 72a0115 to 4905b84 Compare February 14, 2026 02:48
@cottsay cottsay force-pushed the cottsay/rapid-iteration branch from 4905b84 to 2dc3019 Compare February 14, 2026 02:48
Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

For changes related to rosdep:

  • ✅ New rosdep keys are named appropriately
  • ✅ Platforms for new rosdep rules are valid
  • ✅ New keys appear suitable for rosdep

For changes related to yamllint:

  • ✅ All new lines of YAML pass linter checks

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

For changes related to rosdep:

  • ✅ New rosdep keys are named appropriately
  • ✅ Platforms for new rosdep rules are valid
  • ✅ New keys appear suitable for rosdep

For changes related to yamllint:

  • ✅ All new lines of YAML pass linter checks

@cottsay cottsay closed this Feb 14, 2026
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