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: Update scripts for windows comapatibiltiy #405

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

Conversation

Nikhilkumarr-dev
Copy link

@Nikhilkumarr-dev Nikhilkumarr-dev commented Mar 9, 2025

Title: "fix: Update scripts for Windows compatibility"

Description :

Description

Fixed test scripts to run properly on Windows systems. The changes include:
--Added cross-env for the cross-platform environment variables
--Fixed ESlint path issues for Windows
--Updated test scripts JSON formatting for Windows compatibility

Problem

The test scripts were failing on Windows due to:
--Unix-style environment variable setting
--Path formatting issues with ESLint
--JSON parsing errors in test scripts

Solution

  • Added cross-env package for cross-platform compatibility
  • Updated ESLint path format
  • Fixed JSON string escaping in test scripts

Testing

  • All 180 tests passing on Windows system
  • Verified lint and test commands work correctly
  • Maintained compatibility with Unix systems

Related Issue

Related to #394
Screenshot 2025-03-09 170513

--Add cross-env for the cross-platform environment variables
--Fix ESlint path issues for Windows
--Update test scripts to work on Windows systems
Copy link

google-cla bot commented Mar 9, 2025

Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

View this failed invocation of the CLA check for more information.

For the most up to date status, view the checks section at the bottom of the pull request.

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