-
Notifications
You must be signed in to change notification settings - Fork 2.3k
DISABLE_ESLINT_PLUGIN: command not found #5134
Comments
Hi @swisstackle, truffle dashboard uses craco to configure the CRA scripts. See their website for more information. |
@cds-amal I just installed craco globally (6.4.3), but I am still getting the same error. Technically that shouldnt make a difference anyways, because without having it installed globally, it should just use the craco that is being installed by yarn in the dashboard package. |
Thanks, @swisstackle. Please let me know what you discover. |
@cds-amal I think that a lot of issues stem from me using windows.
and deleted the corresponding "SKIP_PREFLIGHT_CHECK=true" and "DISABLE_ESLINT_PLUGIN=true" from the package.json. I obviously won't commit those but I will keep it locally for development purposes. Other issues I've encountered were the precommit stuff (husky) that I just commented out for now (no nvm for windows). But I will have to find a different solution for that as it is important for the checks (I assume) when I create pull requests. |
I think we should make the build work on Windows. We want people to be able to contribute code regardless of what OS they use. @swisstackle do you happen to have a WSL install? If so, would you mind testing whether or not your changes build correctly under WSL in addition to Windows? If they do, I'd be really happy if you'd be willing to submit those changes as a PR to unbreak the build for Windows users. |
I've just reproduced the same error on a different (almost new) windows system. So yes, most likely Windows is the problem. Husky functionality and the disable_entlint thing. Everything else worked when building. Yes, WSL is installed. Here's one problem with committing the changes:
Also it should be mentioned to either use Git Bash or WSL for the build (Bash scripts). Ill add that in the contribute.md file. |
Yeah, you're correct about the husky pre-commit hooks, but otherwise we'd be super happy for anything that you'd be willing to contribute that would otherwise improve the state of things on Windows. That said there is an internal effort happening to bring about a Windows CI build, so that will probably improve things pretty dramatically when that drops. |
@benjamincburns I can make a PR for the .env file if you wish. It resolved the Dashboard issue I've had. |
That'd be awesome - thanks! |
…to .env file for Windows compatibility
|
Hello all
'DISABLE_ESLINT_PLUGIN' is not recognized as an internal or external command, |
I figured I would transfer this problem to github issues because I dont believe that thats what the discussions are for.
Issue
I am currently trying to build all packages on my local machine to contribute to truffle:
When yarn/lerna (yarn bootstrap) tries to build the truffle dashboard package, I am getting the error 'DISABLE_ESLINT_PLUGIN' is not recognized as an internal or external command, which is used in the package.json to run react-scripts without eslint.
According to react-scripts, this tag was added with the version 4.0.2, but the package.json in the dashboard package references version ^4.0.0 for react-scripts. This is a little bit confusing to me, because how could you guys build the dashboard package with this tag, if you guys use version ^4.0.0? But then again, the carrot makes sure that the latest version is installed, so the issue might clearly be on my side.
Here is the PR that added the 'DISABLE_ESLINT_PLUGIN' tag to react-scripts: facebook/create-react-app#10170
Steps to Reproduce
Expected Behavior
All packages are being built
Actual Results
Error: ['DISABLE_ESLINT_PLUGIN' is not recognized as an internal or external command]
Environment
truffle version
): latest develop statusnode --version
): 16.14npm --version
): 8.10.0The text was updated successfully, but these errors were encountered: