Skip to content

Commit 9781850

Browse files
Riccardo Cipolleschikelset
Riccardo Cipolleschi
authored andcommitted
Restore envinfo for test_windows (#38062)
Summary: We had to disable the envinfo command on test_windows to get the CI back to green. The reason why it started failing is because they released 7.9.0 which does not seem to have the executable on Windows, so `npx` fails to find what to run. This fix restore the command in a way that it should display the envinfo using an older version of the package. I also added a task to come back to this periodically. ## Changelog: [Internal] - Restore envinfo on windows Pull Request resolved: #38062 Test Plan: - CircleCI: test_windows stays green Reviewed By: cortinico Differential Revision: D47016995 Pulled By: cipolleschi fbshipit-source-id: 368367caed7ea49d7419475580a39f9406c54757 # Conflicts: # .circleci/config.yml
1 parent 21daa6e commit 9781850

File tree

1 file changed

+7
-1
lines changed

1 file changed

+7
-1
lines changed

.circleci/config.yml

Lines changed: 7 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1046,9 +1046,15 @@ jobs:
10461046
name: Enable Yarn with corepack
10471047
command: corepack enable
10481048

1049+
# it looks like that, last week, envinfo released version 7.9.0 which does not works
1050+
# with Windows. I have opened an issue here: https://github.com/tabrindle/envinfo/issues/238
1051+
# TODO: T156811874 - Revert this to npx envinfo@latest when the issue is addressed
10491052
- run:
10501053
name: Display Environment info
1051-
command: npx envinfo@latest
1054+
command: |
1055+
npm install -g envinfo
1056+
envinfo -v
1057+
envinfo
10521058
10531059
- restore_cache:
10541060
keys:

0 commit comments

Comments
 (0)