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

Improve Bash script readability and add color-coded output #346

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

mostafasoufi
Copy link
Contributor

This PR improves the existing Bash script by making it more human-readable and providing better user interaction through the following changes:

  • Color-Coded Output: Utilizes tput to introduce colored feedback for errors, progress updates, and successful operations.

    • Red for errors
    • Green for successful operations
    • Yellow for warnings or skipped steps
    • Cyan for informational progress messages
    • Improved Readability: Added descriptive echo statements at key steps (e.g., downloading files, creating databases, etc.) to
    • make the script's execution flow clear and easy to follow.
  • Progress Feedback: Each major step (WordPress installation, test suite setup, and database management) now provides clear and distinct feedback to the user.

  • Confirmation for Database Recreation: The script prompts the user to confirm before deleting and recreating an existing test database, ensuring safer operation.

This enhancement will significantly improve the user experience when running the script by making the outputs more readable and informative.

Screenshot 2024-09-18 at 10 01 36 AM Screenshot 2024-09-18 at 9 56 19 AM

- Added color-coded feedback using `tput` for better user interaction
- Included progress messages to display each step clearly (e.g., downloading, database creation)
- Added confirmation prompt before deleting and recreating the database
- Improved overall script readability with organized `echo` statements
@mostafasoufi mostafasoufi requested a review from a team as a code owner September 18, 2024 07:09
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