-
Notifications
You must be signed in to change notification settings - Fork 442
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
Build Error (broccoli-persistent-filter:EslintValidationFilter) in mirage/config.js #1399
Comments
@samselikoff I have no idea if this is mirage-specific so feel free to close if it doesn't seem relevant. I really just wanted to open it in case we forget and start googling the error again. 🎈 |
Very bizarre - thanks for reporting! Going to be tackling Mirage bugs next week so will look then 👍 |
I'm having the same issue here with a brand new Ember app using eslint. 🤚 |
workaround: hold |
@ezpuzz Thank you – that seemed to fix @beerlington's issue for me as well! I didn't fully understand the discussion over there... is a more "permanent" fix coming in the form of an updated (Sidenote – Prettier is awesome and seems to work well with JS files in Ember! Why have I not been using it?!) |
For now, should we add some Prettier install instructions to Mirage's docs? Trying to figure out what Mirage's responsibility is here |
Thanks to @beerlington for sharing this fix: miragejs/ember-cli-mirage#1399
Looks like that paths RFC got merged for Ember CLI, so there's a longer-term fix on the way. For now I just added some documentation to the Install step that warns Prettier users to either pin the version or add the line to the ignore file: Closing for now, thanks everyone! |
We need to pin `eslint-plugin-prettier` to 2.6.0 See following github issue for reason why: miragejs/ember-cli-mirage#1399 Regarding prettierrc-configuration. We will use same config as `emberjs/ember.js` for constistency.
We need to pin `eslint-plugin-prettier` to 2.6.0 See following github issue for reason why: miragejs/ember-cli-mirage#1399 Regarding prettierrc-configuration. We will use same config as `emberjs/ember.js` for constistency.
We need to pin eslint-plugin-prettier to 2.6.0 for now See the following issue for reference: miragejs/ember-cli-mirage#1399
We need to pin eslint-plugin-prettier to 2.6.0 for now See the following issue for reference: miragejs/ember-cli-mirage#1399
Hey! Our team has run into this a few times and my google skills are failing me so I wanted to just open an issue in case anyone else runs into this.
Basically after installing mirage in a barebones ember app using eslint, it will no longer boot.
We get the following error:
I have a public repo where you can see the issue - https://github.com/beerlington/mirage-build-error
The first commit in the repo boots the app fine. The second commit adds mirage and then no longer boots.
The fix for this is to add the following to
.eslintignore
:and then the server boots.
Just for search purposes, the full stack trace of the error is:
The text was updated successfully, but these errors were encountered: