Skip to content

Fixed configuration of Prettier and env variable setting for Windows #12

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

Merged

Conversation

giuliopulina
Copy link
Contributor

Changes:

  • using cross-env to set env variables across platforms
  • set Prettier options "endOfLine": "auto" to make sure that EOL is treated correctly

I reverted some previous change on redis-cluster configuration, because I'm not sure that setting "IP=0.0.0.0" env variable is safe for everyone.
Currently, my tests on Windows 10 don't work without that setting, but I don't want to risk to break others' work just because it's not working for me.

@darrachequesne darrachequesne merged commit 310026e into socketio:main Feb 16, 2024
@darrachequesne
Copy link
Member

Awesome, thanks 👍

@giuliopulina giuliopulina deleted the feature/make_test_work_on_windows branch February 16, 2024 09:44
@darrachequesne
Copy link
Member

The CI failures are unrelated to this change, it seems some tests are indeed randomly failing. Looking into this...

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.

2 participants