Skip to content
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.

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

Adding bats as a dependency #1

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Conversation

aaron-roller-anybotics
Copy link

@aaron-roller-anybotics aaron-roller-anybotics commented Sep 27, 2024

Please add the following dependency to the rosdep database.

Package name:

Bash Automated Testing System (bats or bats-core)

Package Upstream Source:

https://github.com/bats-core/bats-core?tab=readme-ov-file

Purpose of using this:

Automated testing for bash scripts. Useful to validate any bash scripts that may be used instead of the standard python scripts.

Distro packaging links:

Links to Distribution Packages

https://bats-core.readthedocs.io/en/stable/installation.html

Please Add This Package to be indexed in the rosdistro.

ROSDISTRO bats

The source is here:

https://github.com/bats-core/bats-core

Checks

  • All packages have a declared license in the package.xml
  • [x ] This repository has a LICENSE file
  • This package is expected to build on the submitted rosdistro

@aaron-roller-anybotics
Copy link
Author

@samuelba This is just a practice pull request to see if it makes sense for bats to be included. Per this it may not?

I will make the pull request to the master repository if this passes your test.
Notice only 1 checkbox was checked. package.xml does not exist so the license requirement doesn't match. package.json does have a mention of the MIT license.

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