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

Fixes syntax for checking if pre-commit is installed in isaaclab.sh #1422

Merged
merged 8 commits into from
Dec 5, 2024
2 changes: 1 addition & 1 deletion isaaclab.sh
Original file line number Diff line number Diff line change
Expand Up @@ -328,7 +328,7 @@ while [[ $# -gt 0 ]]; do
fi
# run the formatter over the repository
# check if pre-commit is installed
if [ ! command -v pre-commit &>/dev/null ]; then
if ! command -v pre-commit &>/dev/null; then
echo "[INFO] Installing pre-commit..."
pip install pre-commit
fi
Expand Down
2 changes: 1 addition & 1 deletion source/extensions/omni.isaac.lab/config/extension.toml
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
[package]

# Note: Semantic Versioning is used: https://semver.org/
version = "0.27.18"
version = "0.27.19"

# Description
title = "Isaac Lab framework for Robot Learning"
Expand Down
11 changes: 10 additions & 1 deletion source/extensions/omni.isaac.lab/docs/CHANGELOG.rst
Original file line number Diff line number Diff line change
@@ -1,5 +1,14 @@
Changelog
---------

0.27.19 (2024-12-05)
~~~~~~~~~~~~~~~~~~~~

Fixed
^^^^^

* Fixed the condition in `isaaclab.sh` that checks whether `pre-commit` is installed before attempting installation.

kellyguo11 marked this conversation as resolved.
Show resolved Hide resolved


0.27.18 (2024-12-04)
~~~~~~~~~~~~~~~~~~~~
Expand Down