Skip to content

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
@schlessera
Copy link
Member

Thanks for the work on this, @mostafasoufi !

I'm not merging this right now, as we're currently in discussions about the future of this script in general, and whether we might want to switch the entire testing flow over to something like wp-env.

I'll keep you updated in here.

@mostafasoufi
Copy link
Contributor Author

Sounds good! Happy to help if needed. @schlessera

swissspidy added a commit that referenced this pull request Oct 8, 2025
Incorporating changes from #346
@swissspidy
Copy link
Member

I incorporated these changes in #356

Still not decided how we proceed with this script though.

@swissspidy swissspidy closed this Nov 2, 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.

3 participants