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: add dev environment setup check script #13

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

Conversation

therealharshit
Copy link
Member

Fixes #12

Fix Details

  1. Improved Error Handling: Now gracefully handles cases where commands are not found or fail to execute.
  2. Version Extraction with Regex: Ensures robust parsing of version numbers, preventing format mismatches.
  3. Better Readability and Maintainability: Refactored function names and logic for clarity.
  4. Streamlined Success Handling: Uses all() to determine overall script success more cleanly.

@therealharshit therealharshit changed the title fis: add dev environment setup check script fix: add dev environment setup check script Feb 13, 2025
Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi! Thanks for opening this pull request.

Because this is your first time contributing to this repository, make sure you've read our Contributor Guide and Code of Conduct.

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.

Issue: Dev Environment Setup Check Issue
1 participant