Skip to content

Conversation

@MiracleInvoker
Copy link

@MiracleInvoker MiracleInvoker commented Dec 25, 2025

Contributor License Agreement

This repository (optuna-dashboard) and Goptuna share common code.
This pull request may therefore be ported to Goptuna.
Make sure that you understand the consequences concerning licenses and check the box below if you accept the term before creating this pull request.

  • I agree this patch may be ported to Goptuna by other Goptuna contributors.

Reference Issues/PRs

None

What does this implement/fix? Explain your changes.

This PR fixes a failure in the Dev Container setup script caused by the use of pushd and popd commands in a script running under #!/bin/sh.

The Problem:
The .devcontainer/postCreateCommand.sh script uses pushd and popd, which are Bash extensions and not defined in the POSIX sh standard. This causes the script to fail on systems where /bin/sh is a strict POSIX shell (like dash), preventing the environment from building.

The Fix:
Replaced pushd and popd with standard cd commands. This allows the script to run successfully using the default #!/bin/sh interpreter without requiring a dependency on bash.

@MiracleInvoker MiracleInvoker changed the title Fix devcontainer shell interpreter to support pushd/popd Replace bash-specific pushd/popd with POSIX cd in devcontainer script Dec 25, 2025
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