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

fix: CLI::promptByMultipleKeys() and prompt() #8873

Merged
merged 6 commits into from
May 13, 2024

Commits on May 8, 2024

  1. test: call PhpStreamWrapper::restore() as soon as possible

    If it is not called, assertions may not work correctly.
    kenjis committed May 8, 2024
    Configuration menu
    Copy the full SHA
    573676c View commit details
    Browse the repository at this point in the history
  2. docs: fix typos in comment

    kenjis committed May 8, 2024
    Configuration menu
    Copy the full SHA
    7e918ba View commit details
    Browse the repository at this point in the history
  3. test: add test cases

    kenjis committed May 8, 2024
    Configuration menu
    Copy the full SHA
    35a295c View commit details
    Browse the repository at this point in the history
  4. Configuration menu
    Copy the full SHA
    56f9c2d View commit details
    Browse the repository at this point in the history
  5. fix: CLI::prompt() may incorrectly return default value

    See CLITest::testPromptInputZero().
    kenjis committed May 8, 2024
    Configuration menu
    Copy the full SHA
    94703ba View commit details
    Browse the repository at this point in the history
  6. Configuration menu
    Copy the full SHA
    e772b9d View commit details
    Browse the repository at this point in the history