Skip to content

Add a new pip rule#36

Closed
cottsay wants to merge 3 commits into
rosdistro-reviewerfrom
cottsay/dismissal-test
Closed

Add a new pip rule#36
cottsay wants to merge 3 commits into
rosdistro-reviewerfrom
cottsay/dismissal-test

Conversation

@cottsay

@cottsay cottsay commented Jun 18, 2026

Copy link
Copy Markdown
Owner

No description provided.

@cottsay cottsay self-assigned this Jun 18, 2026

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown

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
  • ✅ New keys appear suitable for rosdep
  • 📝 Disclaimer: Pip-only rules cannot be used by packages on the ROS buildfarm

For changes related to yamllint:

  • ✅ All new lines of YAML pass linter checks

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown

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
  • ✅ New keys appear suitable for rosdep
  • 📝 Disclaimer: Pip-only rules cannot be used by packages on the ROS buildfarm

For changes related to yamllint:

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

Comment thread rosdep/python.yaml Outdated
aaa-pip:
'*':
pip:
packages: [aaa]

Copy link
Copy Markdown

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

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown

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
  • ✅ New keys appear suitable for rosdep
  • 📝 Disclaimer: Pip-only rules cannot be used by packages on the ROS buildfarm

For changes related to yamllint:

  • ✅ All new lines of YAML pass linter checks

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.

1 participant